-
Notifications
You must be signed in to change notification settings - Fork 12
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
opendap/hyrax_ncwms:1.16.2 wms broken #30
Comments
I think issue lies in the netcdf-java library used by NcWMS 4.2.2. That (older) library fails to properly URL encode the constraint expressions sent to the DAP service. Uncoded URLs that contain characters that should be encoded (like "[" and "]") have been rejected by Tomcat since roughly June 2018 (Tomcat-8.0.53). Certainly the version of Tomcat in the "latest" docker container will reject them, returning only an HTTP status of 400 and no response body. See this issue for more: #29 The NcWMS-2.2.2 bundle contains I think I need to reach out to our friends at NcWMS and see if they have a newer version with an updated netcdf-java library. Sincerely, Nathan |
Thanks Nathan. It had crossed my mind that might be related. FWIW with TDS I've been working around this with relaxedQueryChars as documented in https://tomcat.apache.org/tomcat-8.5-doc/config/http.html (and more specifically described in many posts/questions/answers). I can probably moderate the issue by crafting a custom server.xml for tomcat until the upstream problem can be solved - or I keep holding off on upgrading hyrax. If I try tweaking tomcat, I'll first try: Gareth BTW. Our data providers and users place high value on WMS (via ncWMS) and also godiva3, particularly to get a quick overview of data. I'm considering running ncWMS2 in a separate container and figuring out how to integrate that service in to hyrax. I see there is a container referenced in ncWMS2 docs: https://reading-escience-centre.gitbooks.io/ncwms-user-guide/content/02-installation.html#docker |
Hi Gareth, I reached out to the ncWMS folks and maybe they'll supply a new WAR file with an update netcdf-java...
This section of our documentation: https://opendap.github.io/hyrax_guide/Master_Hyrax_Guide.html#WMS_Service |
Hi Gareth, I heard back from Guy Griffiths - looks like we need to update the ncWMS distribution in the server. I need to get together with my team to decide how to proceed, but I suspect it means a new numbered release, as:
What is your level of urgency? (I suspect fairly high based on your comments about how popular this featured is) Sincerely, Nathan |
Thanks Nathan. Sounds good. My level of urgency is low. I'm just sticking with 1.16.1 for the moment and don't think there are any critical updates I need. It will be good to have this resolved in future releases along with general developments/improvements. |
Any news? |
bump |
Thanks Gareth - We will be planning the next block of work during the next ten days and I will get this on the schedule. |
any news? |
While working on #36 I built a new container that should be equivalent to the one in dockerhub (and his issue). Something is different and I get the following stacktrace info. Possibly I should make a new issue. Ultimately I'd like to help with upstream fixes and wait. for 1.16.3 :-)
|
Rebuilt and pushed |
@gajowi Please let me know if this works for you when you have a chance. Thanks, N |
Will do.
…On Sat, 19 Dec 2020, 6:33 am Nathan Potter, ***@***.***> wrote:
@gajowi <https://github.com/gajowi> Please let me know if this works for
you when you have a chance.
Thanks,
N
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#30 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAUSKFIT2PQWYW3BHK2TH4LSVOVCJANCNFSM4OYBAOBQ>
.
|
Hi Nathan. I have has partial success testing. Actually that is optimistic... I have a new error: javax.servlet.ServletException: ncWMS configuration object is null (when getting /ncWMS2/wms/lds/testdata/nc/coads_climatology.nc?SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.3.0) |
I am trying to run the Docker setup provided herein for 1.16.3 (using docker-compose). Hyrax works fine, but ncWMS does not and I get messages like Could not initialize class com.sun.xml.bind.v2.model.impl.RuntimeBuiltinLeafInfoImpl. Is this connected to this issue? I can see that 1.16.3 still runs netcdf4-5.0.0-beta6.jar? |
Use 1.16.3 |
Something has broken that I do not understand, and unfortunately other folks have been contributing and I am afraid a change has slipped through that broke this.
Well, yes and no. But insofar as ncWMS is still broken then yes.
No part of Hyrax utilizes that jar, it is part of the ncWMS2-2.4.2 war file so I have no control over its inclusion. |
I will try to revisit the issue this week and see what I can learn |
@gajowi @steingod I am curious to know: Do you get the same error if you run ncWMS-2.4.2 standalone? |
I haven't tried that yet, but will check (unfortunately it may take some days before I have a time slot to push on that again). |
I have also tried building and running just the ncWMS docker container:
And there I did encounter the error that @gajowi and @steingod encountered when I tried to go the to top of the service:
This sure seems to be an issue with ncWMS because that docker image contains only tomcat-8 and ncWMS-2.4.2 and not Hyrax. Interestingly, my browser remembered things and it coughed up this URL:
Which fails in two ways, both expected at this point:
But, interestingly, the page manages to load and does not generate a Hey @guygriffiths, do you have any insights regarding these two problems?? The two problems:
|
@ndp-opendap thanks for helping out with this. I can confirm that i was able to make the system work when I built the bundled container. I have the same issues as you though concerning background maps. I can however now, continue testing the service configuration set up with THREDDS Catalogues which was what I intended to do. I will follow any progress here on whether the docker-compose setup will start working again. Thanks. |
@ndp-opendap - The background map issue is due to an issue with our hosting. The university is going through some infrastructure changes and the background maps at godiva.reading.ac.uk are not always available. Hopefully this will become more stable soon (although I just checked and it appears to be working right now). If the problem persists, you should be able to select the Demis or the GEBCO background maps which are hosted elsewhere. The XML class not found issue I'm not sure about, but I believe may be related to Java version changes - we certainly had issues with moving towards Java 11 being the default version. ncWMS 2.5.0 fixes these issues, so if you're able to update to that it might help things. Note that v2.5.0 requires Java 11 as a minimum. |
Hi @guygriffiths, We are encountering the
Or on a docker image based on:
Migrating these deployments to Java 11 will take us some time. It would be great to find a work around in the meantime. Do you have any ideas? Thanks Nathan |
I'm not sure, I'd probably try rolling back a couple of versions, maybe to 2.3.1 |
ncWMS Godiva3 viewer is not working with the current DockerHub Image: https://hub.docker.com/r/opendap/hyrax_ncwms Tag: However, Godiva is working with version: Tested using the following URI: After loading the container via: I have not investigated the differences that may have caused this issue, but I thought it could use an update after I found myself here with a similar problem 3 years later with a workaround. |
Thanks for the heads up @mylesmc123 ! Glad you found a workaround for this annoying issue. On docker hub, It looks like @jgallagher59701 @ndp-opendap any thoughts on how to correct this? |
Previously wms and godiva3 worked for/with /opendap/data/nc/coads_climatology.nc.html
https://data-cbr.csiro.au/ncWMS2/Godiva3.html?server=https://data-cbr.csiro.au/ncWMS2/wms/lds/testdata/nc/coads_climatology.nc (also fine without tls...).
With 1.16.2 I think the related wms GetCapabilities response is OK but when getting data there is an error. 400 Bad Request
Could not read underlying data
(for url /ncWMS2/wms/lds/data/nc/coads_climatology.nc?FORMAT=image%2Fpng&TRANSPARENT=TRUE&STYLES=default-scalar%2Fdefault&LAYERS=lds%2Fdata%2Fnc%2Fcoads_climatology.nc%2FSST&TIME=0000-12-16T01%3A20%3A06.000Z&COLORSCALERANGE=%2C&NUMCOLORBANDS=250&ABOVEMAXCOLOR=extend&BELOWMINCOLOR=extend&BGCOLOR=extend&LOGSCALE=false&SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&SRS=EPSG%3A4326&BBOX=-57.12,32.88,65.76,155.76&WIDTH=256&HEIGHT=256)
I haven't fully understood the associated godiva3 set of requests. The ones that return empty tiles seem to be OK. The error only happens for tiles with data.
Can you see the problem?
regards, Gareth
The text was updated successfully, but these errors were encountered: