sansmoraxz commented on PR #4055: URL: https://github.com/apache/iggy/pull/4055#issuecomment-5620514448
OK I rebased and went with a different approach now. There's one dedicated ID reserved for external users (which should not collide with local users) and is configurable (but not persisted). Default value if external users enables - `u32::MAX`. Server boot should crash if ID collision detected at startup and not after server is booted. For external users the main identifiers are unique strings. Also the auth flow is now `local -> external` instead of external first. This prevents the potential lag of normal iggy hitting and getting failures on the auth server and then falling back to normal. Both single node and cluster modes are validated to work without issues. _Side note:_ `segment_recovery` tests are currently broken at master which I rebased this from. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
