Skip to content

AuthN Hot Fixes#113

Merged
inFocus7 merged 19 commits intoagentregistry-dev:mainfrom
inFocus7:authn-fixes-v2
Feb 24, 2026
Merged

AuthN Hot Fixes#113
inFocus7 merged 19 commits intoagentregistry-dev:mainfrom
inFocus7:authn-fixes-v2

Conversation

@inFocus7
Copy link
Copy Markdown
Collaborator

@inFocus7 inFocus7 commented Jan 28, 2026

Description

Hot fixes for authn integration

Changes

  • Added path skipping to certain paths to prevent API AuthN middleware checks on them (e.g. /ping).
  • Created new ErrCLINoStoredToken error, to avoid CLI AuthN checks blocking commands when not logged into registry (e.g. init).
  • Added callback for token management
    • e.g. this goes through authn and has access token, then this token can easily be configured in the extension side (if needed) using the callback. good for cases where extension needs the token without having to re-authn on its end
  • Updated CLIAuthn to be factory-based
  • Rename push to publish to keep verbiage in sync.

metadata

/kind cleanup
NONE

Signed-off-by: Fabian Gonzalez <[email protected]>
@inFocus7 inFocus7 changed the title Authn Hot Fixes AuthN Hot Fixes Jan 28, 2026
@inFocus7 inFocus7 marked this pull request as ready for review February 23, 2026 23:17
Signed-off-by: Fabian Gonzalez <[email protected]>
Signed-off-by: Fabian Gonzalez <[email protected]>
@inFocus7 inFocus7 merged commit 82fc642 into agentregistry-dev:main Feb 24, 2026
5 checks passed
christian-posta pushed a commit to christian-posta/agentregistry that referenced this pull request Mar 9, 2026
# Description

Hot fixes for authn integration

## Changes

- Added path skipping to certain paths to prevent API AuthN middleware
checks on them (e.g. /ping).
- Created new `ErrCLINoStoredToken` error, to avoid CLI AuthN checks
blocking commands when not logged into registry (e.g. init).
- Added callback for token management
- e.g. this goes through authn and has access token, then this token can
easily be configured in the extension side (if needed) using the
callback. good for cases where extension needs the token without having
to re-authn on its end
- Updated CLIAuthn to be factory-based
- Rename `push` to `publish` to keep verbiage in sync.

**metadata**
```
/kind cleanup
```
```release-note
NONE
```

---------

Signed-off-by: Fabian Gonzalez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants