Skip to content

Fix navbar auth display and cross-domain login#290

Merged
bokelley merged 2 commits into
mainfrom
bokelley/fix-topnav-login
Dec 12, 2025
Merged

Fix navbar auth display and cross-domain login#290
bokelley merged 2 commits into
mainfrom
bokelley/fix-topnav-login

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

  • Fixed navbar showing login buttons even when user is authenticated by adding cache-control headers to /api/config endpoint
  • Added cross-domain redirects for auth-requiring pages on adcontextprotocol.org to agenticadvertising.org, since session cookies are domain-scoped to AAO
  • Simplified nav.js to use identical AAO branding across all sites

Changes

  • Prevent browser caching of /api/config response with no-store headers
  • Redirect /dashboard, /team, /onboarding, /member-profile on AdCP domain to AAO
  • Redirect /auth/login on AdCP domain to AAO with proper return_to handling
  • Make navbar identical everywhere with AAO logo always linking to agenticadvertising.org

🤖 Generated with Claude Code

bokelley and others added 2 commits December 11, 2025 23:09
- Add cache-control headers to /api/config to prevent browser caching of auth state, which was causing navbar to show login buttons after successful authentication
- Redirect auth-requiring pages on adcontextprotocol.org to agenticadvertising.org since session cookies are domain-scoped to AAO
- Make navbar identical across all sites with AAO branding and always show AdCP link
- Redirect /auth/login on AdCP domain to AAO to ensure cookies persist across domains

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bokelley bokelley merged commit 9de5bfd into main Dec 12, 2025
5 checks passed
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.

1 participant