Skip to content

[TOMEE-2201] NPE when injecting Application#135

Merged
asfgit merged 3 commits intoapache:masterfrom
tveronezi:TOMEE-2201_NPE_when_injecting_Application
Jul 17, 2018
Merged

[TOMEE-2201] NPE when injecting Application#135
asfgit merged 3 commits intoapache:masterfrom
tveronezi:TOMEE-2201_NPE_when_injecting_Application

Conversation

@tveronezi
Copy link
Member

There are two things here:

  • the javax.ws.rs.core.Application option was not in the if chain in JndiEncBuilder.java.
  • MapObjectReference.java was throwing NPE if the referenced map was null.

Help on creating unit tests for these guys is needed.

@tveronezi tveronezi changed the title TOMEE-2201 NPE when injecting Application [TOMEE-2201] NPE when injecting Application Jul 16, 2018
Thiago Veronezi added 2 commits July 16, 2018 12:08
@tveronezi
Copy link
Member Author

CustomContext testing is already covered by server/openejb-cxf-rs/src/test/java/org/apache/openejb/server/cxf/rs/CustomContextTest.java

@asfgit asfgit merged commit 9573042 into apache:master Jul 17, 2018
jgallimore pushed a commit to jgallimore/tomee that referenced this pull request Sep 2, 2018
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.

2 participants