Skip to content

Do not recommend using anonymous user, but make name optional#265

Open
timokoessler wants to merge 5 commits intomainfrom
readme-no-anonymous-user
Open

Do not recommend using anonymous user, but make name optional#265
timokoessler wants to merge 5 commits intomainfrom
readme-no-anonymous-user

Conversation

@timokoessler
Copy link
Member

@timokoessler timokoessler commented Mar 6, 2026

Because the same rate limits would apply to all unauthenticated users.

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

⚡ Enhancements

  • Relaxed user constructors to require only non-empty id, allowing empty names

🔧 Refactors

  • Stopped setting anonymous defaults in sample app middleware; conditional user set

More info

Because the same rate limits would apply to all unauthenticated users.
@teta2k teta2k changed the title docs: Do not use an anonymous user Do not recommend using anonymous user, but make name optional Mar 6, 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