Jakob Ankarhem
Jakob Ankarhem
Hey, I have a similar problem. I think the approach klarna has for solving this problem is quite good. They allow you to lock the UI with a suspend function...
@costaht I think you were the one initially suggesting this feature. So maybe you have some input?
Isn't this just a matter of setting the baseurl in next config and doing namespace in express? Can you elaborate on what is hindering this @sct ?
Oh, yeah that's true. So the solution for us would be to prepend every use we have of express, links etc with an env variable we can pass in to...
> Would love to see this implemented. Is this on the roadmap? Well, this is functional. It needs to be reviewed is all.
> I just pulled this and tested it with OIDC from [Authentik](https://github.com/goauthentik/authentik). It works as intended after a few changes: > > * [This regex](https://github.com/ankarhem/overseerr/blob/oidc-support/src/components/Settings/SettingsUsers/index.tsx#:~:text=/%5E(%3F!%3A%5C/%5C/)(%5Ba%2DzA%2DZ0%2D9%2D_%5D%2B%5C.)*%5Ba%2DzA%2DZ0%2D9%5D%5Ba%2DzA%2DZ0%2D9%2D_%5D%2B%5C.%5Ba%2DzA%2DZ%5D%7B2%2C11%7D%3F%24/) is overly restrictive -- it...
Fixed review comments and rebased the branch
> @ayazgoff can you share your Authentik configs, and @ankarhem have the changes @ayazgoff stated been implemented? Yes they were addressed in ef24e30
> Some comments. in server/utils/oidc.ts in the getOIDCRedirectURL, you are making assumptions about the url.pathname. This should be configurable. For example, Authentik uses /application/o/authorize and not /authorize. > Right, should...
@obsidiangroup I have made the changes now (fetches auth endpoint from wellknown configuration and handles closing of the popup), please try again.