Red Hat Bugzilla – Bug 809775
Repository configuration generated by Guvnor uses deprecated Jackrabbit classes
Last modified: 2013-11-07 17:14:07 EST
Created attachment 575087 [details]
Description of problem:
From server log:
> org.apache.jackrabbit.core.security.SimpleAccessManager is deprecated. Please use org.apache.jackrabbit.core.security.simple.SimpleAccessManager instead
> org.apache.jackrabbit.core.security.SimpleLoginModule is deprecated. Please use org.apache.jackrabbit.core.security.simple.SimpleLoginModule instead
> org.apache.jackrabbit.core.persistence.bundle.BundleDbPersistenceManager is deprecated. Please use org.apache.jackrabbit.core.persistence.pool.BundleDbPersistenceManager instead
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. http://localhost:8080/jboss-brms/ > Administration > Repository Configuration
2. choose arbitrary values and generate the config
generated configuration contains deprecated classes (according to what is logged by jackrabbit)
the configuration should contain the classes that replace the deprecated ones
repository seem to be working OK with the deprecated classes
Mistakenly moved to ON_QA, moving back to MODIFIED.
Hi Jervis, on which branch have you made that fix? If it's really fixed I think it can be safely merged into 5.3.x, what do you think?
Hi Jiri, the fix is on 5.3.x , 5.4.x and master branch already.
The fix is commited in 5.3.x branch on April 23
The ER6 's cutting date is April 18. So it is not in product yet.
As Jiri pointed out, there is no risk to include this fix.
So I think I am going to pick this one in ER7, otherwise we would need to revert it in project which I think wouldn't make more sense.
Modified status means to me that developer fixed this issue but not yet picked up by product. Otherwise it would move to ON_QA status. Would the "modified" status mean differently to others?
This need to be set block+ flag before picking in product.
Hello Jervis, is this https://github.com/droolsjbpm/guvnor/commit/441997f559caecc14a0033455a143d1df134c16c is the only commit that has been made in order to fix this issue? Please notice that also classes from org.apache.jackrabbit.core.persistence.bundle package were deprecated and should be replaced with classes from org.apache.jackrabbit.core.persistence.pool.
Please include this update too or correct me if I'm wrong. Changing back to assigned.
Hi Jiri, thanks for the reminding. Fixed.
Please verify them on ER8.
Verified fixed in 5.3.0 ER8. Deprecated classes aren't used in generated configs anymore.