Open brendanheywood opened 8 years ago
Thanks for the feedback!
Nice option to be added and I personally like it.
However, we are in hard work nowadays. If you need that in the short time, please, consider making a PR. Otherwise, we will evaluate this in the middle term.
Thanks a lot!
Jordi
Awesome! Yes we will try and get a PR together, it's a little bit down our list too. We originally tried to get this into core but it was knocked back:
ie the use case if that we don't want to actually authenticate via this plugin, but via some other existing plugin like ldap or saml which don't have an internal password. So we'd configure this plugin to be earlier in the plugin priority list, and then have a new config checkbox something like 'Don't authenticate, just prevent login unless in ip whitelist'