Skip to content

API: Add authorization hook (v2) - #849

Merged
markylaing merged 3 commits into
canonical:v2from
roosterfish:auth_hook_v2
Oct 16, 2026
Merged

API: Add authorization hook (v2)#849
markylaing merged 3 commits into
canonical:v2from
roosterfish:auth_hook_v2

Conversation

@roosterfish

@roosterfish roosterfish commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Backports #840 and #848.

@roosterfish
internal/state/hooks: Add OnAuthenticate hook
7cede4a 
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
(cherry picked from commit dfd7219)
(cherry picked from commit 7223f21)
@roosterfish
rest/access/handlers: Prioritize OnAuthenticate hook over regular mTL…
553494b 
…S check

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
(cherry picked from commit a56e8fa)
(cherry picked from commit 9f2ff6d)
@roosterfish
microcluster/app: Ignore returned value (errcheck)
7516a08 
We only ever send nil into the chan.
Therefore switch to struct{} as its done for ctx.Done().

Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
(cherry picked from commit 4de6ffd)
(cherry picked from commit 9b0625d)
@markylaing
markylaing merged commit 93f9e78 into canonical:v2 Oct 16, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@markylaing markylaing markylaing approved these changes
@MusicDin MusicDin MusicDin approved these changes
@simondeziel simondeziel Awaiting requested review from simondeziel
@tugbataluy tugbataluy Awaiting requested review from tugbataluy
@TimosAmpel TimosAmpel Awaiting requested review from TimosAmpel

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@roosterfish @markylaing @MusicDin