diff --git a/services/web/app/src/Features/BrandVariations/BrandVariationsHandler.js b/services/web/app/src/Features/BrandVariations/BrandVariationsHandler.js index a3c3b452e4..a9c30d07c9 100644 --- a/services/web/app/src/Features/BrandVariations/BrandVariationsHandler.js +++ b/services/web/app/src/Features/BrandVariations/BrandVariationsHandler.js @@ -78,5 +78,5 @@ function setV1AsHostIfRelativeURL(urlString) { // As it only applies if the second argument is not absolute, we can use it to transform relative URLs into // absolute ones using v1 as the host. If the URL is absolute (e.g. a filepicker one), then the base // argument is just ignored - return new URL(urlString, settings.apis.v1.url).href + return new URL(urlString, settings.apis.v1.publicUrl).href } diff --git a/services/web/test/unit/src/BrandVariations/BrandVariationsHandlerTests.js b/services/web/test/unit/src/BrandVariations/BrandVariationsHandlerTests.js index 264be584aa..08491cc511 100644 --- a/services/web/test/unit/src/BrandVariations/BrandVariationsHandlerTests.js +++ b/services/web/test/unit/src/BrandVariations/BrandVariationsHandlerTests.js @@ -26,7 +26,7 @@ describe('BrandVariationsHandler', function () { this.settings = { apis: { v1: { - url: 'http://overleaf.example.com', + publicUrl: 'http://overleaf.example.com', }, }, modules: { @@ -111,7 +111,9 @@ describe('BrandVariationsHandler', function () { '12', (err, brandVariationDetails) => { expect( - brandVariationDetails.logo_url.startsWith(this.settings.apis.v1.url) + brandVariationDetails.logo_url.startsWith( + this.settings.apis.v1.publicUrl + ) ).to.be.true return done() }