-
Notifications
You must be signed in to change notification settings - Fork 74
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
Update to golang v1.22 for builder base #334
Comments
Also intersted in that! |
@mholt Any plans to bump golang in the builder image? Our build also broke after your recent commits in caddy-ratelimit and bumping the required golang to v1.22 (see here). Thanks! |
@brotbert Dependabot broke the build; my follow-up commit fixed the build except for those using old caddy versions apparently. The latest commit actually makes it work for old Caddy versions too: 2d18cd2c76cc5578483450f44f417bca8764a8bc |
@mholt We appreciate the quick feedback on it, but unfortunately it still doesn't work because of the minimum required golang version: https://github.com/mholt/caddy-ratelimit/blob/master/go.mod#L3 See also for reference: https://go.dev/doc/modules/gomod-ref#go |
No, but you can use the 2.8.0-beta.2 builder image which has Go 1.22. But I agree the |
@francislavoie Ah, thanks for the idea. Currently we decided to pin the caddy-ratelimit to the last working commit revision 359636d470b54958bdf331ff9c687192ba348daa. |
We're using Go 1.22 for Caddy v2.8.0 which is releasing now |
I'm attempting to build a docker container with the experimental Tailscale plugin: https://github.com/tailscale/caddy-tailscale
My issue is that trying to build with a Dockerfile like this errors because the version of Golang is v1.21, and v1.22 is required for caddy-tailscale:
The text was updated successfully, but these errors were encountered: