Skip to content

[Portal] Add Authentication Mode selector for Traditional Web Application#5663

Closed
fungc-io wants to merge 18 commits intoauthgear:mainfrom
fungc-io:dev-3469-auth-mode-selector
Closed

[Portal] Add Authentication Mode selector for Traditional Web Application#5663
fungc-io wants to merge 18 commits intoauthgear:mainfrom
fungc-io:dev-3469-auth-mode-selector

Conversation

@fungc-io
Copy link
Copy Markdown
Member

@fungc-io fungc-io commented Apr 16, 2026

Summary

  • Add `x_traditional_webapp_session_type` field (`cookie` | `access_token`) to the OAuth client config schema and Go struct
  • Add an Authentication Mode radio group on the Traditional Web Application detail page (between Basic Info and URIs); Server-side SDK is shown first and is the default for new apps; Cookie Session is preserved as the default for existing apps that have no value stored
  • Add a Select Authentication Mode step in the `/configuration/apps/add` creation wizard, inserted between type selection and save for `traditional_webapp` clients only
  • Update the quickstart guide (both the sidebar widget and the post-creation screen) to always show Next.js and Other Framework links for Traditional Web Application clients
  • Cookie Session mode shows the Cookie-based authentication section; Server-side SDK mode shows the Refresh Token and Access Token fields
  • Inline note shown when switching from Server-side SDK → Cookie Session within an unsaved editing session

ref DEV-3469

🤖 Generated with Claude Code

@tung2744
Copy link
Copy Markdown
Contributor

Some checks are failing

@fungc-io
Copy link
Copy Markdown
Member Author

@tung2744 , let's discuss on this change before merging it

@tung2744
Copy link
Copy Markdown
Contributor

@fungc-io As per latest discussion I think this PR will be closed?

@fungc-io
Copy link
Copy Markdown
Member Author

Yes, I will redesign and re submit

@fungc-io fungc-io closed this Apr 23, 2026
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