Skip to content

docs(auth): fix client_id origin — client generates it, not Hub#41

Open
VasilevNStas wants to merge 1 commit into
APN-Network:masterfrom
VasilevNStas:9-docs/client-id-origin
Open

docs(auth): fix client_id origin — client generates it, not Hub#41
VasilevNStas wants to merge 1 commit into
APN-Network:masterfrom
VasilevNStas:9-docs/client-id-origin

Conversation

@VasilevNStas

@VasilevNStas VasilevNStas commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes the Auth.client_id documentation: the client generates the ID locally (CSPRNG, 9 chars from alphabet ABCDEFGHIJKLMNPQRSTUVWXYZ123456789), the Hub only stores the passed value.

Confirmed by code:

  • ios/APNCommon/LegalPrincipal.swiftSystemRandomNumberGenerator
  • android/.../Principal.ktkotlin.random.Random
  • hub/objects/hub/clients.rbINSERT ... ON CONFLICT DO NOTHING with caller-supplied id

Closes #9

@VasilevNStas

Copy link
Copy Markdown
Contributor Author

@davvd please review

@davvd

davvd commented Jul 4, 2026

Copy link
Copy Markdown
Member

@VasilevNStas we have merge conflicts here, please resolve them and we'll merge

@VasilevNStas
VasilevNStas force-pushed the 9-docs/client-id-origin branch from 5aaf368 to 60cd3de Compare July 4, 2026 18:05
@VasilevNStas

Copy link
Copy Markdown
Contributor Author

@davvd conflicts resolved.

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.

auth.proto says the Hub generates client_id, but the clients generate it locally

2 participants