Set max refresh attempts #1792
Merged
Set max refresh attempts #1792
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bbpennel
approved these changes
Aug 28, 2024
bbpennel
added a commit
that referenced
this pull request
Sep 13, 2024
* BXC-4694 - Handle RangeNotSatisfiable (#1785) * Added new RangeNotSatisfiableException, which gets thrown if the range header exceeds the size of the file. Handle this error with a 416 response * Log headers when an uncaught exception occurs in fedora content controller * Add test for exception * BXC-4667 update datastream controller IT for iiif thumbnails * BXC-4667 cleanup test * BXC-4667 cleanup and logic modifications * BXC-4667 cleaner logic * BXC-4667 handle collection, admin unit, and folder thumbnails that will not use iiif * BXC-4667 update access copies unit test to reflect logic change * BXC-4667 add derivativeContentService test * Set max refresh attempts (#1792) * If end of range is equal to or greater than the size of the file, remove it so we get the remainder of the file. Add more testing for FedoraContentService * Change start of range so that it'll trigger 416 * BXC-4701 - Update clients/tests to solr 9 (#1791) (#1795) * Update to solr 9, and wiremock 3 due to jetty versions. Update some commons usages which were referencing libraries pulled in by old solr versons * Update solr config, make paths absolute since it is now required, and don't double load config * Switch over to wiremock-jre8-standalone to avoid jetty version conflicts with solr * Running solr 9 in docker for the integration tests that ran it in jetty, since it was failing to start. Add solr logging config * Adjusting ghactions * Try solr as docker run * Reenable build * Explicitly add solr-api as compile scope, otherwise solr-core pulls it in as test. Add commons-codec directly in a lower module, since it was previously being imported via transitive dependencies, but no longer is * Clean up of removed performance visualization project (#1797) --------- Co-authored-by: sharonl <[email protected]> Co-authored-by: Dean Farrell <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://unclibrary.atlassian.net/browse/BXC-3156