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

Issue 4208 - Fails to inject SecurityContext into Helloworld-CDI2-SE example #4236

Merged
merged 2 commits into from
Aug 29, 2019

Conversation

pa314159
Copy link
Contributor

No description provided.

@pa314159 pa314159 changed the title Issues/4208 Fails to inject SecurityContext into Helloworld-CDI2-SE example Aug 28, 2019
@pa314159 pa314159 force-pushed the issues/4208 branch 2 times, most recently from c699bb2 to fbd0bfb Compare August 28, 2019 12:07
@jansupol
Copy link
Contributor

Please update copyright year in the next push

@pa314159
Copy link
Contributor Author

@jansupol, rebase or new commit?

@jansupol
Copy link
Contributor

rebase is fine, small PR

@pa314159 pa314159 changed the title Fails to inject SecurityContext into Helloworld-CDI2-SE example Issue 4208 - Fails to inject SecurityContext into Helloworld-CDI2-SE example Aug 28, 2019
@pa314159 pa314159 force-pushed the issues/4208 branch 3 times, most recently from 569ec12 to 3a9430b Compare August 28, 2019 17:35
The method JersetBean.getBeanClass is used to compute the bean
identifier. The default id for a JerseyBean<T> is
"java.lang.Object#jersey", causing collisions of bridge instances.

In particular, for HelloWorldResourceTest, Weld's MapBeanStore returned
a SupplierInstanceBeanBridge instead of the expected SupplierBeanBridge,
because both shared the same bean identifier.

Signed-off-by: pappy <[email protected]>
Copy link
Contributor

@senivam senivam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work :)

This was referenced Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants