grails / grails-spring-security-core

Grails Spring Security Core Plugin
Apache License 2.0
261 stars 223 forks source link

Feature Request: Automatically configure "concurrent sessions" support #562

Open joemccall86 opened 6 years ago

joemccall86 commented 6 years ago

See https://docs.spring.io/spring-security/site/docs/current/reference/html5/#concurrent-sessions

I've implemented this in our Grails application using a combination of resources.groovy and BootStrap.groovy using clientRegisterFilter, but it seems like it would fit right in with this plugin as a configuration option.

I'm thinking something like grails.plugin.springsecurity.trackConcurrentSessions: true|false. It can be false by default.

I'm happy to contribute this feature as a PR if you think it's worthwhile.

ddelponte commented 6 years ago

That's great! ✨ All contributions are welcome!

All I ask is that you provide tests where applicable/possible and be open to a code review on the PR.

Thank-you very much!