Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(gateway): add F3GetCertificate & F3GetLatestCertificate to gateway #12778

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Dec 12, 2024

Only applies a rate limiting cost of "basic" which is 1 token because I believe these are pretty efficient calls.

Doesn't apply a lookback time limit, you can fetch any arbitrary instance's certificate from a public gateway. go-f3's "certstore" is keyed by instance ID which is what's being queried here.

@rvagg rvagg requested a review from masih December 12, 2024 11:16
@rvagg
Copy link
Member Author

rvagg commented Dec 12, 2024

Updated with an itest that makes sure these don't misbehave when it's not running or is disabled. It's a bit too complex to combine the awkward F3 itest setup with the awkward gateway itest setup to test it running. I started, but the F3 itest setup is already huge without pulling more bits in.

itests/gateway_test.go Outdated Show resolved Hide resolved
@rvagg rvagg enabled auto-merge (squash) December 12, 2024 11:43
@rvagg rvagg merged commit e7db584 into master Dec 12, 2024
83 checks passed
@rvagg rvagg deleted the rvagg/f3-gateway branch December 12, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ☑️ Done (Archive)
Development

Successfully merging this pull request may close these issues.

3 participants