Skip to content
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

Grails 7: add the spring-security plugins' versions to grails-bom #13832

Open
jamesfredley opened this issue Nov 12, 2024 · 3 comments
Open

Grails 7: add the spring-security plugins' versions to grails-bom #13832

jamesfredley opened this issue Nov 12, 2024 · 3 comments
Assignees

Comments

@jamesfredley
Copy link
Contributor

Issue description

https://repo.grails.org/ui/repos/tree/PomView/libs-snapshots-local/org/grails/grails-bom/7.0.0-SNAPSHOT/grails-bom-7.0.0-20241112.082000-137.pom

@codeconsole
Copy link
Contributor

codeconsole commented Nov 12, 2024

sure, spring security is already part of the Spring bom, so it would make sense to have the Grails plugin versions

@jamesfredley
Copy link
Contributor Author

jamesfredley commented Nov 12, 2024

Here is the list:
org.grails.plugins:spring-security-cas:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-core:7.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest-gorm:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest-grailscache:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest-memcached:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest-redis:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-rest-testapp-profile:6.0.0-SNAPSHOT
org.grails.plugins:spring-security-ui:5.0.0-SNAPSHOT

org.grails.plugins:grails-redis:5.0.0-SNAPSHOT - used by spring-security-rest-redis

this one is waiting on 2 PRs and the initial snapshot has not been published.
org.grails.plugins:spring-security-acl:5.0.0-SNAPSHOT
grails/grails-spring-security-acl#44
grails/grails-spring-security-acl#45

@jamesfredley jamesfredley changed the title Grails 7: should we add the spring-security plugins' versions to grails-bom Grails 7: add the spring-security plugins' versions to grails-bom Nov 12, 2024
@bkoehm
Copy link
Contributor

bkoehm commented Nov 12, 2024

I have no objection but I do want to point out that spring-security-cas is probably not widely used. UC Berkeley may be one of the few institutions still using this plugin (although I don't know for sure). I'm not sure how well this plugin is going to be maintained over the long term and I don't know if maintenance of the plugin factors into your decision about whether to include it in the BOM or not. I suppose the easy answer is it can later be removed from the BOM if the plugin is not keeping pace with Grails development.

I also opened grails-plugins/grails-spring-security-ldap#27 back on September 23rd for grails-spring-security-ldap but it hasn't been looked at by anyone.
We are also using this plugin, but again, I speculate that UC Berkeley is about the only ones still using this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

5 participants