Skip to content

Update Terms of Service and Privacy Policy for SSO#952

Open
ngnijland wants to merge 1 commit intomasterfrom
update-terms-and-privacy-policy-for-sso
Open

Update Terms of Service and Privacy Policy for SSO#952
ngnijland wants to merge 1 commit intomasterfrom
update-terms-and-privacy-policy-for-sso

Conversation

@ngnijland
Copy link
Collaborator

Summary

  • Terms of Service (Article 8): Added SSO authentication language — Users may authenticate via their organization's identity provider; RocketSim is not responsible for third-party credential management; SSO requires a valid license and domain mapping.
  • Privacy Policy: Added new "Single Sign-On (SSO) Authentication" section covering data collected (email, name, verification status, SSO flag, ID token), the sso_session_code cookie, temporary session data (state, nonce, PKCE verifier), and retention periods.
  • Privacy Policy (Third parties): Added Auth0 (by Okta) as a third-party service used for SSO authentication brokering.
  • Updated privacy statement revision date to 7 March 2026.

Test plan

  • Review legal language in Terms Article 8.1 and new Article 8.3
  • Review Privacy Policy SSO section for accuracy against implementation
  • Review Auth0 third-party disclosure
  • Verify pages render correctly with npm run dev
  • Confirm build passes with npm run build

🤖 Generated with Claude Code

Add SSO-related language to the Terms (Article 8) covering identity
provider-based authentication and responsibility boundaries. Add a new
SSO Authentication section to the Privacy Policy covering data collection,
session data, cookies, and Auth0 as a third-party service.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ngnijland ngnijland force-pushed the update-terms-and-privacy-policy-for-sso branch from 5c826f4 to 1b943a4 Compare March 7, 2026 10:17
@ngnijland ngnijland requested a review from AvdLee March 7, 2026 10:17
Copy link
Owner

@AvdLee AvdLee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants