-
Notifications
You must be signed in to change notification settings - Fork 0
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
Basic implementation with HTTP/GHA cache protocols, OIDC and Expr #1
Conversation
group.GET("/*", cache.get) | ||
group.HEAD("/*", cache.head) | ||
group.POST("/*", cache.put) | ||
group.PUT("/*", cache.put) | ||
group.DELETE("/*", cache.delete) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need some kind of authorization/authentication here? Do we have it somewhere else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have it somewhere else?
We have it as a middleware:
chacha/internal/server/server.go
Line 88 in 7f37e5b
auth.Middleware(server.issToProvider), |
Some things still left to be implemented:
attempt to read fallback cache keys and not just the first oneadd a command-line variable tochacha run
with the external base URLneeded by the GHA cache to do a proper redirectGHA cache protocol testResolves https://github.com/cirruslabs/internal-planning/issues/43.