Browse Source

Require https callback in production, allow for http otherwise

pull/4023/head
Rory Powell 5 years ago
parent
commit
eb8d857864
  1. 4
      packages/worker/src/api/controllers/admin/auth.js

4
packages/worker/src/api/controllers/admin/auth.js

@ -144,7 +144,9 @@ async function oidcStrategyFactory(ctx, configId) {
const chosenConfig = config.configs.filter(c => c.uuid === configId)[0] const chosenConfig = config.configs.filter(c => c.uuid === configId)[0]
const callbackUrl = `${ctx.protocol}://${ctx.host}/api/admin/auth/oidc/callback` // require https callback in production
const protocol = process.env.NODE_ENV === "production" ? "https" : "http"
const callbackUrl = `${protocol}://${ctx.host}/api/admin/auth/oidc/callback`
return oidc.strategyFactory(chosenConfig, callbackUrl) return oidc.strategyFactory(chosenConfig, callbackUrl)
} }

Loading…
Cancel
Save