Skip to content

Conversation

@matrei
Copy link
Contributor

@matrei matrei commented Oct 14, 2025

Closes #1181

Copy link
Contributor

@bkoehm bkoehm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a few more of these in https://apache.github.io/grails-spring-security/7.0.x/core-plugin/guide/index.html with a search string of springsecurity.interceptUrlMap = [.

@matrei
Copy link
Contributor Author

matrei commented Oct 14, 2025

I see a few more of these in https://apache.github.io/grails-spring-security/7.0.x/core-plugin/guide/index.html with a search string of springsecurity.interceptUrlMap = [.

Isn't springsecurity.interceptUrlMap also a thing? Do you think we should replace them all?

@bkoehm
Copy link
Contributor

bkoehm commented Oct 14, 2025

I see a few more of these in https://apache.github.io/grails-spring-security/7.0.x/core-plugin/guide/index.html with a search string of springsecurity.interceptUrlMap = [.

Isn't springsecurity.interceptUrlMap also a thing? Do you think we should replace them all?

Good question. I am not sure. I assumed interceptUrlMap had been replaced, but perhaps not. grails.plugin.springsecurity.web.access.intercept.InterceptUrlMapFilterInvocationDefinition exists. I am not sure when someone would use grails.plugin.springsecurity.interceptUrlMap instead of grails.plugin.springsecurity.controllerAnnotations.staticRules. The documentation may explain this but I haven't dug into the docs further on this.

I'll change my review to Approve.

@matrei
Copy link
Contributor Author

matrei commented Oct 14, 2025

I am not sure.

😄 Same here! Let's merge this one and come back to this later if needed.

@matrei matrei merged commit ae1acf9 into 7.0.x Oct 14, 2025
11 of 12 checks passed
@matrei matrei deleted the issue-1181 branch October 14, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Grails 7 - grails-spring-security Doc - interceptUrlMap

2 participants