Purge with surrogate key not working

[neurosnap]

Greetings, purging with surrogate keys is not currently working for me. It looks like the key values are getting stored URL encoded and then being sent to the Storer as-is and then not finding the keys to clear:

curl -v -H "Authorization: Basic xxx" https://pgs.test/souin-api/souin/surrogate_keys
{"erock-test":",GET-https-erock-test.pgs.test-%2F"}

curl -v -XPURGE -H "Authorization: Basic xxx" -H "surrogate-key: erock-test" https://pgs.test/souin-api/souin/
Status: 204

curl -v -H "Authorization: Basic xxx" https://pgs.test/souin-api/souin/
["GET-https-erock-test.pgs.test-/"]

curl -v -H "Authorization: Basic xxx" https://pgs.test/souin-api/souin/surrogate_keys
{"erock-test":",GET-https-erock-test.pgs.test-%2F"}

https://github.com/darkweak/souin/blob/master/pkg/surrogate/providers/common.go#L223

It looks like we might need to expand the QueryUnescape logic? https://github.com/darkweak/souin/blob/master/pkg/surrogate/providers/common.go#L85

[neurosnap]

This probably isn't the "right" way to fix it but this is what we did inside pico (we are using our own impl of the souin API): https://github.com/picosh/pico/blob/706b21492f4133c7e5519a0a82f9d8c89fed8317/pgs/web.go#L315

[darkweak]

Hey @neurosnap sorry for the long delay. I'll try to write a fix for that.

[darkweak]

@neurosnap you can try the fix using

xcaddy build --with github.com/darkweak/souin/plugins/caddy@cae85eea6558e65bf2eb049a4c885a648fd781d7 --with github.com/darkweak/souin@cae85eea6558e65bf2eb049a4c885a648fd781d7