XWIKI-21763: Disable the server-side image resizing while exporting to PDF

xwiki-platform-core/xwiki-platform-configuration/xwiki-platform-configuration-default/src/main/java/org/xwiki/configuration/internal/AllConfigurationSource.java

@@ -31,6 +31,7 @@
 /**
  * Composite Configuration Source that looks in the following sources in that order:
  * <ul>
+ * <li>execution context</li>
  * <li>user preferences wiki page</li>
  * <li>space preferences wiki page</li>
  * <li>wiki preferences wiki page</li>
@@ -58,10 +59,15 @@ public class AllConfigurationSource extends CompositeConfigurationSource impleme
     @Named("user")
     private ConfigurationSource userPreferencesSource;
 
+    @Inject
+    @Named("executionContext")
+    private ConfigurationSource executionContextSource;
+
     @Override
     public void initialize() throws InitializationException
     {
         // First source is searched first when a property value is requested.
+        addConfigurationSource(this.executionContextSource);
         addConfigurationSource(this.userPreferencesSource);
         addConfigurationSource(this.documentsPreferencesSource);
         addConfigurationSource(this.xwikiPropertiesSource);

xwiki-platform-core/xwiki-platform-configuration/xwiki-platform-configuration-default/src/main/java/org/xwiki/configuration/internal/DefaultConfigurationSource.java

@@ -31,6 +31,7 @@
 /**
  * Composite Configuration Source that looks in the following sources in that order:
  * <ul>
+ * <li>execution context</li>
  * <li>documents sources (space first and then wiki preferences wiki pages)</li>
  * <li>xwiki properties file (xwiki.properties)</li>
  * </ul>
@@ -52,10 +53,15 @@ public class DefaultConfigurationSource extends CompositeConfigurationSource imp
     @Named("documents")
     private ConfigurationSource documentsSource;
 
+    @Inject
+    @Named("executionContext")
+    private ConfigurationSource executionContextSource;
+
     @Override
     public void initialize() throws InitializationException
     {
         // First source is looked up first when a property value is requested.
+        addConfigurationSource(this.executionContextSource);
         addConfigurationSource(this.documentsSource);
         addConfigurationSource(this.xwikiPropertiesSource);
     }

xwiki-platform-core/xwiki-platform-configuration/xwiki-platform-configuration-default/src/test/java/org/xwiki/configuration/internal/AllConfigurationSourceTest.java

@@ -19,6 +19,10 @@
  */
 package org.xwiki.configuration.internal;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.mockito.Mockito.when;
+
 import javax.inject.Named;
 
 import org.junit.jupiter.api.Test;
@@ -27,10 +31,6 @@
 import org.xwiki.test.junit5.mockito.InjectMockComponents;
 import org.xwiki.test.junit5.mockito.MockComponent;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertNull;
-import static org.mockito.Mockito.when;
-
 /**
  * Validate {@link AllConfigurationSource}.
  * 
@@ -51,6 +51,10 @@ class AllConfigurationSourceTest
     @Named("user")
     private ConfigurationSource userPreferencesSource;
 
+    @MockComponent
+    @Named("executionContext")
+    private ConfigurationSource executionContextSource;
+
     @InjectMockComponents
     private AllConfigurationSource configuration;
 
@@ -73,5 +77,10 @@ void getProperty()
         when(this.userPreferencesSource.getProperty("key")).thenReturn("userPreferencesSource");
 
         assertEquals("userPreferencesSource", this.configuration.getProperty("key"));
+
+        when(this.executionContextSource.containsKey("key")).thenReturn(true);
+        when(this.executionContextSource.getProperty("key")).thenReturn("executionContextSource");
+
+        assertEquals("executionContextSource", this.configuration.getProperty("key"));
     }
 }

xwiki-platform-core/xwiki-platform-configuration/xwiki-platform-configuration-default/src/test/java/org/xwiki/configuration/internal/DefaultConfigurationSourceTest.java

@@ -19,6 +19,10 @@
  */
 package org.xwiki.configuration.internal;
 
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.mockito.Mockito.inOrder;
+import static org.mockito.Mockito.when;
+
 import javax.inject.Named;
 
 import org.junit.jupiter.api.Test;
@@ -28,9 +32,6 @@
 import org.xwiki.test.junit5.mockito.InjectMockComponents;
 import org.xwiki.test.junit5.mockito.MockComponent;
 
-import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.mockito.Mockito.*;
-
 /**
  * Unit tests for {@link DefaultConfigurationSource}.
  *
@@ -51,6 +52,10 @@ class DefaultConfigurationSourceTest
     @Named("xwikiproperties")
     private ConfigurationSource xwikiPropertiesSource;
 
+    @MockComponent
+    @Named("executionContext")
+    private ConfigurationSource executionContextSource;
+
     @Test
     void containsKey()
     {
@@ -60,10 +65,12 @@ void containsKey()
         assertTrue(this.source.containsKey("key"));
 
         // Verify that the call order is correct
-        InOrder inOrder = inOrder(xwikiPropertiesSource, this.documentsSource);
+        InOrder inOrder = inOrder(xwikiPropertiesSource, this.documentsSource, this.executionContextSource);
         // First call
-        inOrder.verify(this.documentsSource).containsKey("key");
+        inOrder.verify(this.executionContextSource).containsKey("key");
         // Second call
+        inOrder.verify(this.documentsSource).containsKey("key");
+        // Third call
         inOrder.verify(xwikiPropertiesSource).containsKey("key");
     }
 }

xwiki-platform-core/xwiki-platform-export/xwiki-platform-export-pdf/xwiki-platform-export-pdf-api/src/main/java/org/xwiki/export/pdf/internal/job/PDFExportJob.java

@@ -23,12 +23,14 @@
 import java.io.InputStream;
 import java.net.URL;
 import java.util.List;
+import java.util.Map;
 
 import javax.inject.Inject;
 import javax.inject.Named;
 import javax.inject.Provider;
 
 import org.xwiki.component.annotation.Component;
+import org.xwiki.configuration.TemporaryConfigurationExecutor;
 import org.xwiki.export.pdf.PDFExportConfiguration;
 import org.xwiki.export.pdf.PDFPrinter;
 import org.xwiki.export.pdf.internal.RequiredSkinExtensionsRecorder;
@@ -74,12 +76,19 @@ public class PDFExportJob extends AbstractPDFExportJob
     @Inject
     private PrintPreviewURLBuilder printPreviewURLBuilder;
 
+    @Inject
+    private TemporaryConfigurationExecutor temporaryConfigurationExecutor;
+
     @Override
     protected void runInternal() throws Exception
     {
         if (!this.request.getDocuments().isEmpty()) {
             this.requiredSkinExtensionsRecorder.start();
-            render(this.request.getDocuments());
+            this.temporaryConfigurationExecutor.call("executionContext",
+                Map.of("rendering.imageDimensionsIncludedInImageURL", false), () -> {
+                    render(this.request.getDocuments());
+                    return null;
+                });
             if (!this.status.isCanceled()) {
                 this.status.setRequiredSkinExtensions(this.requiredSkinExtensionsRecorder.stop());
             }

xwiki-platform-core/xwiki-platform-export/xwiki-platform-export-pdf/xwiki-platform-export-pdf-api/src/test/java/org/xwiki/export/pdf/internal/job/PDFExportJobTest.java

@@ -24,6 +24,7 @@
 import static org.junit.jupiter.api.Assertions.assertSame;
 import static org.junit.jupiter.api.Assertions.fail;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
@@ -35,12 +36,16 @@
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
+import java.util.concurrent.Callable;
 
+import javax.inject.Inject;
 import javax.inject.Named;
 
 import org.apache.commons.lang3.StringUtils;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.xwiki.configuration.TemporaryConfigurationExecutor;
 import org.xwiki.export.pdf.PDFExportConfiguration;
 import org.xwiki.export.pdf.PDFPrinter;
 import org.xwiki.export.pdf.internal.RequiredSkinExtensionsRecorder;
@@ -93,6 +98,9 @@ class PDFExportJobTest
     @MockComponent
     private PrintPreviewURLBuilder printPreviewURLBuilder;
 
+    @MockComponent
+    private TemporaryConfigurationExecutor temporaryConfigurationExecutor;
+
     private DocumentReference firstPageReference = new DocumentReference("test", "First", "Page");
 
     private DocumentRenderingResult firstPageRendering = new DocumentRenderingResult(this.firstPageReference,
@@ -129,6 +137,13 @@ void configure() throws Exception
         this.request.setDocuments(
             Arrays.asList(this.firstPageReference, this.secondPageReference, thirdPageReference, fourthPageReference));
 
+        when(this.temporaryConfigurationExecutor.call(eq("executionContext"),
+            eq(Map.of("rendering.imageDimensionsIncludedInImageURL", false)), any())).thenAnswer(invocation -> {
+                @SuppressWarnings("unchecked")
+                Callable<Void> callable = (Callable<Void>) invocation.getArgument(2);
+                return callable.call();
+            });
+
         when(this.authorization.hasAccess(Right.VIEW, this.aliceReference, this.firstPageReference)).thenReturn(true);
         when(this.authorization.hasAccess(Right.VIEW, this.aliceReference, this.secondPageReference)).thenReturn(true);