Skip to content

Adding CF claims and HMAC signing verification - #16

Merged
dawid-nowak merged 4 commits into
mainfrom
users/dawid.nowak/adding_additional_token_verifiaction_methods_to_match_CF
May 19, 2026
Merged

Adding CF claims and HMAC signing verification#16
dawid-nowak merged 4 commits into
mainfrom
users/dawid.nowak/adding_additional_token_verifiaction_methods_to_match_CF

Conversation

@dawid-nowak

Copy link
Copy Markdown
Contributor

Adding HMAC signing and CF

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
@dawid-nowak
dawid-nowak requested a review from lucarlig May 18, 2026 10:30

@lucarlig lucarlig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some concerns

Comment thread crates/contextforge-gateway-rs-lib/src/common.rs
Comment thread crates/contextforge-gateway-rs-lib/src/lib.rs
Comment thread crates/contextforge-gateway-rs-lib/src/layers/claims_id.rs Outdated
Comment thread crates/contextforge-gateway-rs-lib/src/layers/claims_id.rs
Comment thread crates/contextforge-gateway-rs-lib/src/user_config_store/redis_config_store.rs Outdated
Comment thread crates/contextforge-gateway-rs-lib/src/layers/claims_id.rs Outdated
Comment thread crates/contextforge-gateway-rs-lib/Cargo.toml
Comment thread crates/contextforge-gateway-rs-lib/src/layers/claims_id.rs Outdated
Comment thread crates/contextforge-gateway-rs-lib/src/layers/claims_id.rs Outdated
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
@dawid-nowak
dawid-nowak requested a review from lucarlig May 18, 2026 21:11
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
@dawid-nowak
dawid-nowak force-pushed the users/dawid.nowak/adding_additional_token_verifiaction_methods_to_match_CF branch from 44e1acb to b22cd3c Compare May 18, 2026 21:11

@lucarlig lucarlig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dawid-nowak
dawid-nowak merged commit 090e2b9 into main May 19, 2026
6 checks passed
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