firewall icon indicating copy to clipboard operation
firewall copied to clipboard

Fix redirect functionality for redirect_non_whitelisted_config value

Open mrgswift opened this issue 6 years ago • 2 comments

I have been using this firewall package for a while. Thank you!!! Today for the first time, I had a use case where I needed to assign a url to redirect_non_whitelisted_to in my firewall config. After doing so, I noticed the redirect was not working for non-whitelisted users. It returned the http response code defined in the responses.whitelist config value but was completely ignoring the redirect_non_whitelisted_to config value.

I investigated further and noticed a refactor at commit 075b69b, which appears to have removed the condition to check the value of redirect_non_whitelisted_to. I refactored the Whitelist filter function a little bit to fix this.

mrgswift avatar Sep 20 '19 22:09 mrgswift

Bump. Just wondering if you have taken a look at this yet. Thanks!

mrgswift avatar Oct 04 '19 17:10 mrgswift

@mrgswift Have you by chance found a similar package that is actively maintained?

bennettblack avatar May 01 '21 02:05 bennettblack