Red Hat Bugzilla – Bug 1018269
Add a warning that the ejb-security-interceptor quickstart use private API
Last modified: 2014-01-12 19:22:50 EST
Description of problem:
The ejb-security-interceptors quickstart was introduced in EAP 6.1.1. It uses private classes.
A new public API was introduced in 6.2 that broke this quickstart.
Per Brian Stansberry's suggestion:
* They will restore the 2 classes and commit to maintain them for 6.x
* They will not mark the modules public because they aren't and this doesn't justify changing that
* They willwe come up with a plan for a proper API in WF and 6.3 and we deprecate these once we do.
We need to add a warning to the quickstart that it uses a private API that will be deprecated and replaced in EAP 6.3.
Some suggestions from Pete:
"The two classes XXX are part of the <JBoss EAP> private API. They will be maintained for the duration of EAP 6.x. We plan to introduce a public API in EAP 6.3, and encourage you to update to the public API when available. Check the EAP 6.3 release notes for info on the public API"
they can *use* it
you should add to the statement i wrote "Note that the current API (two class names) will be deprecated in EAP 6.3, but continue to be available during the EAP 6.x series
Version-Release number of selected component (if applicable):
Steps to Reproduce:
Created against the wrong product.
Added this note in the readme:
This quickstart uses two classes, org.jboss.as.controller.security.SubjectUserInfo and org.jboss.as.domain.management.security.RealmUser, that are part of the JBoss EAP private API. A public API will become available in the EAP 6.3 release and the private classes will be deprecated, but these classes will be maintained and available for the duration of the EAP 6.x release cycle.
Added a comment for each import:
// The following class is part of the JBoss private API and will be deprecated in the next release.
Fixed in this pull: https://github.com/jboss-developer/jboss-eap-quickstarts/pull/678
Also reverted back to the private API calls.
Fixes are in this commit:
Verified on EAP 6.2 ER6