#EXUI-2656: passport js rollout needs serviceOverride flag

hmcts · Dec 19, 2024 · 813373b · 813373b
1 parent 0bbdbd0
commit 813373b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@hmcts/rpx-xui-node-lib",
-  "version": "2.29.7",
+  "version": "2.29.7-exui-2656-rc1",
   "description": "Common nodejs library components for XUI",
   "main": "dist/index",
   "types": "dist/index.d.ts",

diff --git a/src/auth/models/authOptions.interface.ts b/src/auth/models/authOptions.interface.ts
@@ -16,6 +16,7 @@ export interface AuthOptions {
     allowRolesRegex?: string
     useCSRF?: boolean
     routeCredential?: RouteCredential
+    serviceOverride?: boolean
 }
 
 export interface RouteCredential {

diff --git a/src/auth/models/strategy.class.ts b/src/auth/models/strategy.class.ts
@@ -36,6 +36,7 @@ export abstract class Strategy extends events.EventEmitter {
         allowRolesRegex: '.',
         useCSRF: true,
         routeCredential: undefined,
+        serviceOverride: false,
     }
 
     protected constructor(strategyName: string, router: Router, logger: XuiLogger = getLogger('auth:strategy')) {
@@ -69,6 +70,7 @@ export abstract class Strategy extends events.EventEmitter {
             customHeaders: Joi.any(),
             allowRolesRegex: Joi.string(),
             useCSRF: Joi.bool(),
+            serviceOverride: Joi.bool(),
             routeCredential: Joi.any(),
         })
         const { error } = schema.validate(options)

diff --git a/src/auth/oidc/models/openid.class.ts b/src/auth/oidc/models/openid.class.ts
@@ -130,7 +130,10 @@ export class OpenID extends AuthStrategy {
         const issuer = await this.discoverIssuer()
 
         const metadata = issuer.metadata
-        metadata.issuer = this.options.issuerURL
+        if (!this.options.serviceOverride) {
+            this.logger.log('setting options issuerURL', this.options.issuerURL)
+            metadata.issuer = this.options.issuerURL
+        }
 
         this.logger.log('discover metadata', metadata)