Skip to content

feat: Code review and get back on track - #228

Open
batleforc wants to merge 9 commits into
mainfrom
desloppify/code-health
Open

feat: Code review and get back on track#228
batleforc wants to merge 9 commits into
mainfrom
desloppify/code-health

Conversation

@batleforc

Copy link
Copy Markdown
Owner

Great help of claude code to help me review where i was and estimate what i did wrong

Signed-off-by: Max Batleforc maxleriche.60@gmail.com

last work on the project i need some help to get back on track

Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
Signed-off-by: Max Batleforc <maxleriche.60@gmail.com>
let correlation_id = query_param(&upstream_auth_url, "state");

// Signed with a nonce that does not match the one `/oauth/authorize` sent upstream.
let id_token = sign_id_token(&idp.uri(), "proxyauthk8s", "alice-sub", "wrong-nonce");
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