Add AuthMethod parameter to login page to allow for automatic authentication from external sources #3347
Conversation
src/frontend/src/lib/utils/openID.ts
Outdated
| sessionStorage.setItem("openid_state", state); | ||
| sessionStorage.setItem("openid_nonce", options.nonce); | ||
| window.location.href = authURL.toString(); | ||
| return new Promise(() => {}); // never resolves |
There was a problem hiding this comment.
this is purely to please TypeScript, ultimately the users url is changed, so this will never get hit anyway, unless for some weird strange reason window.location.href fails O.o
I wasn't able to test on Safari, because Apple outright refuse to allow me to download it on Windows :) However, we are not using a popup, so I see no reason why this would fail - as it is instead a redirect - it works on Chrome and Edge. If you can test on Safari that would be appreciated :) |
src/frontend/src/routes/(new-styling)/auth-redirect/+page.svelte
Outdated
Show resolved
Hide resolved
| {#if provider} | ||
| <p class="text-text-primary">Redirecting you to {provider?.name}</p> | ||
| {/if} |
There was a problem hiding this comment.
In svelte, is the #if provider not enough to negate the need for the optional chaining on provider?.name?
Co-authored-by: edenchazard <[email protected]>
…o db-enable-auth-method-param
…o db-enable-auth-method-param
Motivation
Caffeine.AI requires the ability to automatically initiate authentication directly from Caffeine through II by passing a URL parameter that then initiates the flow.
Changes
Tests