-
Notifications
You must be signed in to change notification settings - Fork 0
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
Devmode: NPE before returing found assets from service using the lib #44
Comments
Reproduced independently in two different apps, and seems to happen from webapp.js too - not just services. Effects all kinds of asset types (which all work in prod). |
espen42
pushed a commit
that referenced
this issue
Mar 8, 2021
Problem was that a key under response.headers had a value that was
|
espen42
pushed a commit
that referenced
this issue
Mar 19, 2021
* master: Restore use of testing tool #50 Fixed bug in the path parsing of .get #43: Cleanup, removing redundant logs and comments #43: getCleanPath instead of contextPathOverride and prefix #45: In prod mode, hide (swallow) 400 and 404 error messages. Only output them in XP dev mode. #44: Preventing keys with undefined values in response header. Fixed some tests Removed tests that don't verify lib-static functionality (caching ResourceService now does it) Commented out tests that won't run on existing XP Testing Tool (need to wait for XP 7.7 Testing Tool) Commenting out syntax error in test (which is incomplete anyway) Remove double quotation handling Static root resolution tweak #38: decodeURI solves URI encoding issue. Improving double-quoted etag handling, and charset on error responses. Etag doublequotes handling CR changes Refactored the EtagService, moving some checks out of it to clean the responsibility and dispense with the STATUS and handling - simplifying a lot. Now tests fail, rewriting those. Java implementation can be improved #30 new gradle wrapper Create enonic-gradle.yml # Conflicts: # src/main/resources/lib/enonic/static/index.es6 # src/test/java/lib/enonic/libStatic/etag/EtagServiceTest.java
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In prod mode, after adding a few log lines to src/main/resources/lib/enonic/static/index.es6 # getResponse200:
This produces an image output.
In devmode however, it appears the service itself fails, since it's able to log the output from getResponse200 before failing with a 500:
The text was updated successfully, but these errors were encountered: