Skip to content
This repository has been archived by the owner on Mar 31, 2024. It is now read-only.

Update Go dependencies #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tgeoghegan
Copy link

We are looking at deploying this plugin soon, and so want to update its Go dependencies before we do. I realize that bumping the minimum Go version from 1.12 to 1.21 could be disruptive to existing users of the plugin, so we could consider backing that part of the change out, but I'm not sure if it will be possible to continue supporting that Go version while also keeping dependencies like go-redis and quic-go current.

@francislavoie
Copy link

francislavoie commented Nov 17, 2023

Honestly, I recommend you use https://github.com/pberkel/caddy-storage-redis instead, which is a recently updated/maintained fork which fixes several issues this plugin has (see #42). /cc @pberkel

That said updating dependencies is rarely a bad thing 👍

@tgeoghegan
Copy link
Author

Thank you for that tip, @francislavoie! I landed on this plugin because it's listed among the caddy.storage implementations here. Perhaps the list of endorsed storage plugins should be updated, but I don't see where those module lists get generated in https://github.com/caddyserver/website.

@francislavoie
Copy link

francislavoie commented Nov 17, 2023

Yep, plugin maintainers need to add their own plugins on the website. I asked @pberkel to do so via email so it should show up soon.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants