corazawaf / coraza-caddy

OWASP Coraza middleware for Caddy. It provides Web Application Firewall capabilities
https://www.coraza.io/
Apache License 2.0
330 stars 41 forks source link

chore: upgrade caddy to v2.7 beta #92

Closed jptosso closed 1 year ago

jptosso commented 1 year ago

Fixes go 1.20 compatibility issues. Also, it doesn't affect us directly, as we consume interfaces from caddy. It would affect caddy/main.go though.

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

jcchavezs commented 1 year ago

wonder why you closed it, didn't it work?

On Tue, 18 Jul 2023, 16:27 Juan Pablo Tosso, @.***> wrote:

Closed #92 https://github.com/corazawaf/coraza-caddy/pull/92.

— Reply to this email directly, view it on GitHub https://github.com/corazawaf/coraza-caddy/pull/92#event-9851172990, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXOYAXONVIJBYE2CSSTS3LXQ2MNVANCNFSM6AAAAAA2OPQ6Y4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jptosso commented 1 year ago

I pushed the wrong branch, also it didn't work. I will take a deeper look, I'm sure it worked in some of my experiments