-
-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SAML/SSO #409
Labels
ent: feature
Enterprise features
Comments
See also: plausible/analytics#383 |
@ezekg any news on this feature? |
@clementperon it is on the roadmap (part of #661), but we haven't started work on it yet outside of a POC. Does your team need this feature? |
@ezekg, I try to reduce the usage of account in my company, but we don't have an Enterprise license. |
WorkOS alternative I saw on HN: https://ssoready.com |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Let's use WorkOS for this. Seems pretty simple to integrate. Pricing also looks good.
Endpoints
Redirect
Use the normal create token endpoint. Add an
SSO_REQUIRED
error code when the current account or user requires SSO.The client i.e. Portal MUST manually redirect to the WorkOS authorization URL.
Docs: https://workos.com/docs/reference/sso/get-authorization-url
Callback
The callback is contains a
code
query param that can be exchanged for a user profile. After retrieving the user, we MAY need to create the user, and then we MUST generate a token for the authenticated user.We can optionally use the
state
query param to store additional state, such as account or redirect URL.After the user has been authenticated, we redirect to Portal.
I'm thinking we should generate an SSO-specific token that can only be used via Portal e.g. using a strict/secure cookie.
But using cookies creates other problems, such as accounts using a self-hosted Portal not being able to use SSO. (We COULD keep theIf this becomes a problem, we can manually set aReferrer
in thestate
parameter of the authorization URL, but that introduces an open redirect attack vector...)sso_redirect_url
column for accounts using SSO.Docs: https://workos.com/docs/reference/sso/get-authorization-url/redirect-uri
The text was updated successfully, but these errors were encountered: