[Geoserver-devel] GWC tests failing with no security dispatcher

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

(attachments)

log.txt (10.8 KB)

···

Ian Turton

Ian,

I suspect it might have something to do with the GWC security improvements made by Kevin. I’m currently trying to finish the integration of the GWC rest api upgrade with GeoServer, I’ll keep an I open, and let you know if I come across anything. If by chance I’m able to integrate, I’ll work with Jody to get a PR out because I think this is working in the branch I’m using from Kevin. Unfortunately, Kevin is on holiday this week.

···

On Wed, Sep 6, 2017 at 11:59 AM, Ian Turton <ijturton@anonymised.com> wrote:

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

Ian Turton


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

David Vick

Software Developer | Boundless
dvick@anonymised.com
mobile: +1-636-698-3174

Thanks David, looks like JTS 1.14 upgrade got you going again.

I had a bit of a scare with https://github.com/GeoWebCache/geowebcache/pull/531 failing on build.geoesrver.org but working locally. After building geotools and letting the dependencies seep through GWC now builds.

With that merged travis could finally evaluate - https://github.com/geoserver/geoserver/pull/2508 Which failed … in the YSLD module.

Digging into it there are two test failures in YSLD StyleControlerTest - this is a good test in that it is very brittle of any converters are messed up. I decided to ignore the tests so we would have a working build and we can dig into why those tests are failing tomorrow morning.

I have some early morning meetings and will join you around 9:30 PST, please coordinate on slack and we can get these fixed and the beta release out tomorrow.

···

On 6 September 2017 at 10:14, David Vick <dvick@anonymised.com> wrote:

Ian,

I suspect it might have something to do with the GWC security improvements made by Kevin. I’m currently trying to finish the integration of the GWC rest api upgrade with GeoServer, I’ll keep an I open, and let you know if I come across anything. If by chance I’m able to integrate, I’ll work with Jody to get a PR out because I think this is working in the branch I’m using from Kevin. Unfortunately, Kevin is on holiday this week.


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Jody Garnett

David Vick

Software Developer | Boundless
dvick@anonymised.com
mobile: +1-636-698-3174

On Wed, Sep 6, 2017 at 11:59 AM, Ian Turton <ijturton@anonymised.com> wrote:

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

Ian Turton


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

It might be that but my concern is that it is only occuring on my branch not on master so I suspect it is something I changed in WPS?

Ian

···

On 6 September 2017 at 18:14, David Vick <dvick@anonymised.com.> wrote:

Ian,

I suspect it might have something to do with the GWC security improvements made by Kevin. I’m currently trying to finish the integration of the GWC rest api upgrade with GeoServer, I’ll keep an I open, and let you know if I come across anything. If by chance I’m able to integrate, I’ll work with Jody to get a PR out because I think this is working in the branch I’m using from Kevin. Unfortunately, Kevin is on holiday this week.


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

David Vick

Software Developer | Boundless
dvick@anonymised.com
mobile: +1-636-698-3174

On Wed, Sep 6, 2017 at 11:59 AM, Ian Turton <ijturton@anonymised.com> wrote:

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

Ian Turton


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Ian Turton

Any updates today, or do I need to catch up on gitter.

···

On 7 September 2017 at 00:42, Ian Turton <ijturton@anonymised.com> wrote:

It might be that but my concern is that it is only occuring on my branch not on master so I suspect it is something I changed in WPS?

Ian


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Jody Garnett

On 6 September 2017 at 18:14, David Vick <dvick@anonymised.com> wrote:

Ian,

I suspect it might have something to do with the GWC security improvements made by Kevin. I’m currently trying to finish the integration of the GWC rest api upgrade with GeoServer, I’ll keep an I open, and let you know if I come across anything. If by chance I’m able to integrate, I’ll work with Jody to get a PR out because I think this is working in the branch I’m using from Kevin. Unfortunately, Kevin is on holiday this week.


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Ian Turton

David Vick

Software Developer | Boundless
dvick@anonymised.com
mobile: +1-636-698-3174

On Wed, Sep 6, 2017 at 11:59 AM, Ian Turton <ijturton@anonymised.com> wrote:

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

Ian Turton


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

I’m not sure I got bogged down on GEOS-8215 which is caused by my changes and haven’t had a chance to do a full build since.

Ian

···

On 8 September 2017 at 06:34, Jody Garnett <jody.garnett@…403…> wrote:

Any updates today, or do I need to catch up on gitter.


Jody Garnett

On 7 September 2017 at 00:42, Ian Turton <ijturton@anonymised.com> wrote:

It might be that but my concern is that it is only occuring on my branch not on master so I suspect it is something I changed in WPS?

Ian


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

On 6 September 2017 at 18:14, David Vick <dvick@anonymised.com> wrote:

Ian,

I suspect it might have something to do with the GWC security improvements made by Kevin. I’m currently trying to finish the integration of the GWC rest api upgrade with GeoServer, I’ll keep an I open, and let you know if I come across anything. If by chance I’m able to integrate, I’ll work with Jody to get a PR out because I think this is working in the branch I’m using from Kevin. Unfortunately, Kevin is on holiday this week.


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Ian Turton

David Vick

Software Developer | Boundless
dvick@anonymised.com
mobile: +1-636-698-3174

On Wed, Sep 6, 2017 at 11:59 AM, Ian Turton <ijturton@anonymised.com> wrote:

My branch is currently failing the GWC tests due to an NPE when it tries to call getSecurityDispatcher() in WMSService (which is fair because nothing seems to set that). The problem is the code I was working on is all safely off in gs-core-wps and shouldn’t have any affect on GWC integration.

I’ve attached the log in case that might help but while I see a bunch of warnings none of them look odd.

Has anyone seen something like this before?

Cheers

Ian

Ian Turton


Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot


Geoserver-devel mailing list
Geoserver-devel@anonymised.comrge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Ian Turton