-
Notifications
You must be signed in to change notification settings - Fork 421
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
Broken WMS due to libjpeg-turbo errors in 2.25.0 #653
Comments
@vidlb What CPU architecture are you running on? @NyakudyaA The error in the trace indicates a linkage error, and is because the plugin is pulling in an older version of The Docker image has It looks like the error thrown from here: https://github.com/libjpeg-turbo/libjpeg-turbo/blob/49f3485cb58c3c57a513fe0efe00de7cf319708c/java/org/libjpegturbo/turbojpeg/TJCompressor.java#L146 The line numbers don't match with the current upstream main branch, but do match with The actual C implementation in JNI is one of the functions whose names start with Also, according to the deployment build log, it looks like the first https://github.com/kartoza/docker-geoserver/actions/runs/9140425983/job/25133771692#step:7:285 But then another part of the docker-geoserver/scripts/setup.sh Lines 86 to 94 in 7bcc741
On
So everything is a bit confused; it may only work properly on I think that the solution will be to rebuild While you could roll back to |
My CPU is x86_64 |
Hmm, OK. My theories about library paths (while an issue) are probably not the cause. I've just had a poke at my production instance (running a fork of this image) with GeoServer 2.25.1 on Outputting as PNG is working fine there, but outputting as JPEG is returning the same error you're seeing. I've also tried this out with on The error you're seeing with PNG is while loading a raster JPEG layer, and I don't have any raster layers loaded there to try out. I've done some more digging into the dependency chain, and it looks like GeoSolutions IT's fork of the I've filed the issue upstream (with more detail): geosolutions-it/imageio-ext#305 As for what we could do here, I found removing the My fork has #659 already applied; so I just removed the |
Can you try the 2.25.1 image, we now can disable the plugin if not needed by using the env |
I just tried after adding the empty |
@vidlb you will need to adjust it to become
You will need to adjust the env to skip the default plugin |
All right it seems to work ! |
If |
I find it hard to explain for the empty case, would be simpler if the variable was
Edit: libjpeg-turbo should be discouraged |
What is the bug or the crash?
WMS seems broken in 2.25.0.
Layer preview doesn't always work, on a fresh container with sample raster data.
Steps to reproduce the issue
Start a 2.25.0 container and try to preview any sample raster data, PNG or JPEG.
The WMS returns the following xml for PNG preview of layer "nurc:Img_Sample" :
Here is the full stack trace for this PNG error
For other raster layers, PNG is working but JPEG returns invalid data / firefox cannot render it.
Here is the full stack trace for JPEG
Versions
Latest 2.25.0 image (the one uploaded this morning)
Additional context
With env
ADDITIONAL_JAVA_STARTUP_OPTIONS='-Ddisable.turbojpeg=true'
, the error is gone for most layers (except "nurc:Img_Sample" which seems to be problematic for different reasons).The text was updated successfully, but these errors were encountered: