Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SEAMSECURITY-76 #24

Merged
merged 2 commits into from Aug 12, 2011

Conversation

@maschmid
Copy link
Contributor

commented Aug 11, 2011

Two commits:

The first one changes the configuration of *ScopeProducers from CDI alternatives (which don't work across Bean Archives unless on non-standard-complaint JBoss AS 6 ) to Seam Config. This change is necessary for AS7.

All the *ScopeProducers (such as OpenIdProviderInVirtualApplicationScopeProducer, OpenIdProviderInApplicationScopeProducer, ...) are @Vetoed, and to enable the specific producer, you now have to use, e.g.:

<idp:SamlIdpInApplicationScopeProducer>
<s:modifies/>
</idp:SamlIdpInApplicationScopeProducer>

in seam-beans.xml

The other changes in this commit are necessary to avoid Unresolved and Ambiguous dependencies, this introduces new interfaces (OpenIdRpBeanApi, OpenIdProviderBeanApi, ...) that are used internally to inject the "entity beans" (OpenIdRpBean, OpenIdProviderBean, ...) it is important to always use these when injecting the beans so we ensure that the correct configured producer is used... (I assume this was the original design)

The other commit fixes the examples wrt. these changes and makes them work on AS7.

maschmid added 2 commits Aug 11, 2011
Use @Veto on producers and let them be enabled by seam-config instead of
the CDI Alternative approach, which doesn't work across bean archives
sbryzak added a commit that referenced this pull request Aug 12, 2011

@sbryzak sbryzak merged commit 5035bdc into seam:develop Aug 12, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.