search

gamalan / caddy-tlsredis

Redis Storage using for Caddy TLS Data
Apache License 2.0
95 stars 31 forks source link

Update Go dependencies #53

Open tgeoghegan opened 11 months ago

tgeoghegan commented 11 months ago

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 commented 11 months ago

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 :+1:

tgeoghegan commented 11 months ago

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 commented 11 months ago

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.