handle() adds the OperationList to the RequestContext before calling …

…OperationList.applyNonEndpointMutations() (fixes #589)

CHANGES.md

@@ -3,6 +3,8 @@
 ## 5.0.6
 
 * IIIF information endpoints always return JSON in HTTP 4xx responses.
+* Fixed a bug whereby the values of the `operations` and `page_count` keys
+  in the delegate context were not set.
 * TurboJpegProcessor is able to generate non-JPEG derivative images, which
   fixes an HTTP 415 error that would occur when trying to do that.
 * Fixed a crop-offset bug that could occur when using PdfBoxProcessor to

src/main/java/edu/illinois/library/cantaloupe/resource/ImageRequestHandler.java

@@ -384,10 +384,12 @@ public void handle(OutputStream outputStream) throws Exception {
                 try {
                     fullSize = info.getSize(operationList.getPageIndex());
                     requestContext.setMetadata(info.getMetadata());
-                    operationList.applyNonEndpointMutations(info, delegateProxy);
-                    operationList.freeze();
                     requestContext.setOperationList(operationList, fullSize);
                     requestContext.setPageCount(info.getNumPages());
+                    // This must be done *after* the request context is fully
+                    // populated, as some of the mutations may depend on it.
+                    operationList.applyNonEndpointMutations(info, delegateProxy);
+                    operationList.freeze();
                 } catch (IllegalArgumentException | IndexOutOfBoundsException e) {
                     throw new IllegalClientArgumentException(e);
                 }