Hi guys,

Sorry for being late. I had other things to care lately. I've finally got
time for this. I hope it's not too late, or not too bad of a problem.
I've got a NPE when creating a swagger example. It's very simple to
reproduce. Simply create a jaxrs application and include...

    <dependency>
      <groupId>io.swagger.core.v3</groupId>
      <artifactId>swagger-jaxrs2</artifactId>
      <version>2.0.1</version>
    </dependency>

This is supposed to give you the /openapi.json and /openapi.yaml endpoints.
In our case it gives NPE. I've created this PR which fixes it and adds the
swagger example.

My vote is 0 if you find it's OK to have this until next version. If there
is another way to fix/workaround this without code change, that would be
even better.

https://github.com/apache/tomee/pull/134

[]s,
Thiago.





On Fri, Jul 13, 2018 at 7:29 AM, Alex The Rocker <alex.m3...@gmail.com>
wrote:

> Hello,
>
> Yes Thank you very much Jon for the great TomEE release work!
> I confirm what you wrote.
> Better release 7.0.5 with everything that works.
> Hope to see later a 7.0.6 supporting Java 11 !
>
> Alex
>
> 2018-07-12 12:24 GMT+02:00 Jonathan Gallimore <
> jonathan.gallim...@gmail.com>:
> > I currently have:
> >
> > 1 +1 for the release as it is (Romain)
> > 3 requests for a release with Tomcat 8.5.31 (Gurkan, Felipe, and Alex
> (Alex
> > replied to me directly))
> >
> > Is anyone -1 for a TomEE 7.0.5 release built on Tomcat 8.5.31? Please
> speak
> > up now if you do object so we can talk about it more on the list here.
> From
> > my own perspective, I'm ok with the release as it is, and I'd also +1 a
> > 7.0.5 release with Tomcat 8.5.31 provided it didn't have other issues.
> >
> > I'll look to re-roll the release again later today, unless someone
> objects.
> >
> > Many thanks everyone.
> >
> > Jon
> >
> > On Wed, Jul 11, 2018 at 1:17 AM, Gurkan Erdogdu <cgerdo...@gmail.com>
> wrote:
> >
> >> Hi Jon
> >> Thanks for initiating this.
> >> I opened a bug in Tomcat regarding java:/ namespace and it will be
> >> corrected in 8.5.33 and upper versions. If we distribute the TomEE with
> >> 8.5.32, it will be a problem for users who uses lookups with openejb.
> So,
> >> for this release we can stick to 8.5.31. WDYT?
> >> Regards.
> >> Gurkan
> >>
> >>
> >> On Tue, Jul 10, 2018 at 9:26 PM, Jonathan Gallimore <
> >> jonathan.gallim...@gmail.com> wrote:
> >>
> >> > Hi Everyone,
> >> >
> >> > Here is the second roll of TomEE 7.0.5. Please can you take a look and
> >> > vote? Everyone, committer or not, is encouraged to test and vote.
> >> >
> >> > Staging repo:
> >> > https://repository.apache.org/content/repositories/
> orgapachetomee-1115
> >> >
> >> > Source zip:
> >> > https://repository.apache.org/content/repositories/
> >> > orgapachetomee-1115/org/apache/tomee/tomee-project/7.
> >> > 0.5/tomee-project-7.0.5-source-release.zip
> >> >
> >> > Dist area:
> >> > https://dist.apache.org/repos/dist/dev/tomee/staging-1115/
> >> >
> >> > Legal:
> >> > https://dist.apache.org/repos/dist/dev/tomee/staging-1115/legal.zip
> >> >
> >> > Keys:
> >> > https://dist.apache.org/repos/dist/release/tomee/KEYS
> >> >
> >> > Libraries changed since TomEE 7.0.4:
> >> >
> >> > Tomcat => 8.5.32
> >> > CXF => 3.1.15
> >> > Johnzon => 1.0.1
> >> > OWB => 1.7.5
> >> > XBean => 4.9
> >> > XmlSchema core => 2.2.3
> >> > OpenJPA => 2.4.3
> >> >
> >> > Changes since the last roll:
> >> >
> >> > - Remove javax.xml.soap-api-1.3.5.jar library which was incorrectly
> >> > included
> >> > - Update to Tomcat 8.5.32
> >> > - Change JNDI name used for datasource in CDI TCK test to use an
> >> equivalent
> >> > name under the java: namespace
> >> >
> >> > Changelog:
> >> > https://issues.apache.org/jira/browse/TOMEE-2175?jql=project
> >> > %20%3D%20TOMEE%20AND%20(status%20%3D%20Resolved%20OR%20statu
> >> > s%20%3D%20CLOSED)%20AND%20fixVersion%20%3D%207.0.5%20O
> >> > RDER%20BY%20priority%20DESC%2C%20updated%20DESC
> >> >
> >> > (If anyone knows a better way to get that list, let me know ;-) )
> >> >
> >> > Please vote:
> >> >  +1: Release
> >> >  -1 Do not release because ...
> >> >
> >> > The vote will be open for 3 days or the consensus is binding (At
> least 3
> >> > binding votes).
> >> >
> >> > Many thanks
> >> >
> >> > Jon
> >> >
> >>
>

Reply via email to