Skip to content

Replace dynamic error message with static "Access Denied"#16528

Merged
sjohnr merged 2 commits intospring-projects:mainfrom
kwondh5217:gh-16514
Mar 20, 2025
Merged

Replace dynamic error message with static "Access Denied"#16528
sjohnr merged 2 commits intospring-projects:mainfrom
kwondh5217:gh-16514

Conversation

@kwondh5217
Copy link
Copy Markdown
Contributor

Motivation

  • HttpStatusServerAccessDeniedHandler previously returned the exception message,
    which could expose sensitive information.
  • This PR ensures that the handler always returns a static "Access Denied" message.

Modification

  • Updated HttpStatusServerAccessDeniedHandler to return a static message.
  • Updated tests to verify the response body.

Result

Closes gh-16514

Closes spring-projectsgh-16514

Signed-off-by: Daeho Kwon <trewq231@naver.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 3, 2025
@sjohnr sjohnr self-assigned this Feb 3, 2025
@sjohnr sjohnr added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 3, 2025
Closes spring-projectsgh-16514

Signed-off-by: Daeho Kwon <trewq231@naver.com>
@sjohnr sjohnr merged commit 90dd31c into spring-projects:main Mar 20, 2025
@sjohnr sjohnr added this to the 6.5.0-RC1 milestone Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Send a static message from HttpStatusServerAccessDeniedHandler

3 participants