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

PAYARA-2229 fixed CDI extension injection regression with EAR files #2517

Merged

Conversation

lprimak
Copy link
Contributor

@lprimak lprimak commented Mar 11, 2018

  • Fixed InjectionManager to defer looking up JNDI until it knows it's the correct class

  • CDI injection services are added with the proper BDA/Bundle combindations
    Fixes regressions introduced with PAYARA-1200 and couple of upstream bugs

fixes #2081

@lprimak
Copy link
Contributor Author

lprimak commented Mar 11, 2018

Also tested that PAYARA-1200 and issue #900 still works

@lprimak lprimak changed the base branch from master to Payara4 March 11, 2018 06:34
… the correct class

- CDI injection services are added with the proper BDA/Bundle combindations
Fixes regressions introduced with PAYARA-1200 and couple of upstream bugs
@lprimak lprimak force-pushed the PAYARA-2229-injection-in-second-war branch from 0e0546a to 50acee7 Compare March 11, 2018 07:06
@lprimak
Copy link
Contributor Author

lprimak commented Mar 11, 2018

jenkins quick test payara 4

@payara-ci
Copy link
Contributor

Quick build and test passed!

@lprimak lprimak added this to the Payara 4.x milestone Mar 11, 2018
@Pandrex247
Copy link
Member

Jenkins, quick test Payara 4 JDK7 please

@payara-ci
Copy link
Contributor

Quick build and test passed!

@Pandrex247 Pandrex247 merged commit b6808a4 into payara:Payara4 Mar 22, 2018
@fturizo fturizo removed the 4:Ready label Mar 22, 2018
@lprimak lprimak deleted the PAYARA-2229-injection-in-second-war branch November 28, 2020 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants