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

SEC-3203: GlobalMethodSecurityConfiguration typo in autowiring log message #3402

Closed
spring-issuemaster opened this issue Jan 24, 2016 · 1 comment
Assignees
Milestone

Comments

@spring-issuemaster
Copy link

@spring-issuemaster spring-issuemaster commented Jan 24, 2016

Robert Novotny (Migrated from SEC-3203) said:

GlobalMethodSecurityConfiguration has two typos:

  • method setMethodSecurityExpressionHandler logs a message "Not autwiring PermissionEvaluator since size != 1". However, there are no PermissionEvaluators involved. Instead, method handles MethodSecurityExpressionHandler, which may lead to confusing log message. Furthermore, there is a typo in the message
  • method setPermissionEvaluator logs a message with a typo "Not autwiring PermissionEvaluator since size != 1".
@novotnyr

This comment has been minimized.

Copy link
Contributor

@novotnyr novotnyr commented Aug 16, 2016

  • setPermissionEvaluator typo is obsolete due to changes in bb997ee
novotnyr pushed a commit to novotnyr/spring-security that referenced this issue Aug 16, 2016
Fixes a misleading message that warns about
PermissionEvaluator when MethodSecurityExpressionHandler
should be mentioned instead.

Fixes spring-projectsgh-3402
@rwinch rwinch closed this in #4026 Aug 16, 2016
rwinch added a commit that referenced this issue Aug 16, 2016
Fixes a misleading message that warns about
PermissionEvaluator when MethodSecurityExpressionHandler
should be mentioned instead.

Fixes gh-3402
@rwinch rwinch self-assigned this Aug 16, 2016
@rwinch rwinch added in: config and removed Open in: core labels Aug 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.