-
Notifications
You must be signed in to change notification settings - Fork 186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement an enabled flag for all filters #639
Comments
Is the enable=false for all filters including Spring and Apache Shiro etc or is this also for a Filter Provider? |
I mean it should be possible to do it for each individual filter, but the implementation should be shared. |
@dblock What was your originally thinking on why to switch "Naming wise, I would change disableSSO to enabled"? I got away on other things back in 2018 when that was originally added. It was only recently I started to use some of the newer logic, not this flag exactly but the abiilty for this to always run on windows and for a vintela filter to run on linux. In my case I made both live. That part seemed a bit contensious as well at the time but for cross platform it works beautifully now. So I think based on tickets I'm seeing that there was a left over concern to switch 'disableSSO' to 'enabled'. I do agree that reads better but give the going on 2 year gap I'm not sure if that still makes sense to really do or not. I also have no real insight into how many are actually using the parameter that was added. What I do know is that vintela I if I recall came with a very similar option in naming around disableVsj or something to that effect. |
Yes, I only was commenting on the naming. |
See #638 (comment)
Allow all filters to be disabled with
enabled='false'
.Deprecate the
disableSSO
option that shipped in 1.9.1 via #636.The text was updated successfully, but these errors were encountered: