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

[Authz] Adjusted forbidden message for new security route configuration #195368

Merged

Conversation

elena-shostak
Copy link
Contributor

@elena-shostak elena-shostak commented Oct 8, 2024

Summary

Adjusted forbidden message for new security route configuration to be consistent with ES.

Closes: #195365

@elena-shostak elena-shostak added enhancement New value added to drive a business result Feature:Security/Authorization Platform Security - Authorization Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 8, 2024
@elena-shostak
Copy link
Contributor Author

/ci

@elena-shostak elena-shostak marked this pull request as ready for review October 8, 2024 11:53
@elena-shostak elena-shostak requested a review from a team as a code owner October 8, 2024 11:53
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@azasypkin azasypkin self-requested a review October 9, 2024 08:38
Copy link
Member

@azasypkin azasypkin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@elena-shostak elena-shostak deleted the 195365-authz-forbidden-message branch October 9, 2024 14:36
@elena-shostak elena-shostak restored the 195365-authz-forbidden-message branch October 9, 2024 15:22
@elena-shostak elena-shostak reopened this Oct 9, 2024
@elena-shostak elena-shostak enabled auto-merge (squash) October 10, 2024 08:36
@elena-shostak
Copy link
Contributor Author

@elasticmachine merge upstream

@elena-shostak elena-shostak merged commit 2759994 into elastic:main Oct 10, 2024
21 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11274209996

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 10, 2024
…on (elastic#195368)

## Summary

Adjusted forbidden message for new security route configuration to be
consistent with ES.

__Closes: https://github.com/elastic/kibana/issues/195365__

(cherry picked from commit 2759994)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 10, 2024
…guration (#195368) (#195761)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Authz] Adjusted forbidden message for new security route
configuration (#195368)](#195368)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Elena
Shostak","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-10T12:40:15Z","message":"[Authz]
Adjusted forbidden message for new security route configuration
(#195368)\n\n## Summary\r\n\r\nAdjusted forbidden message for new
security route configuration to be\r\nconsistent with
ES.\r\n\r\n\r\n__Closes:
https://github.com/elastic/kibana/issues/195365__","sha":"2759994e2d53b294a3a049f69bd56fc2e8477e77","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Security","enhancement","release_note:skip","Feature:Security/Authorization","v9.0.0","backport:prev-minor"],"title":"[Authz]
Adjusted forbidden message for new security route
configuration","number":195368,"url":"https://github.com/elastic/kibana/pull/195368","mergeCommit":{"message":"[Authz]
Adjusted forbidden message for new security route configuration
(#195368)\n\n## Summary\r\n\r\nAdjusted forbidden message for new
security route configuration to be\r\nconsistent with
ES.\r\n\r\n\r\n__Closes:
https://github.com/elastic/kibana/issues/195365__","sha":"2759994e2d53b294a3a049f69bd56fc2e8477e77"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195368","number":195368,"mergeCommit":{"message":"[Authz]
Adjusted forbidden message for new security route configuration
(#195368)\n\n## Summary\r\n\r\nAdjusted forbidden message for new
security route configuration to be\r\nconsistent with
ES.\r\n\r\n\r\n__Closes:
https://github.com/elastic/kibana/issues/195365__","sha":"2759994e2d53b294a3a049f69bd56fc2e8477e77"}}]}]
BACKPORT-->

Co-authored-by: Elena Shostak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) enhancement New value added to drive a business result Feature:Security/Authorization Platform Security - Authorization release_note:skip Skip the PR/issue when compiling release notes Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adjust unauthorized response error message
4 participants