Skip to content

Comments

HDDS-14681. [STS] Support StringLike Condition operator in IAM session policy and handle certain errors more gracefully#9795

Merged
sodonnel merged 1 commit intoapache:HDDS-13323-stsfrom
fmorg-git:HDDS-14681
Feb 20, 2026
Merged

HDDS-14681. [STS] Support StringLike Condition operator in IAM session policy and handle certain errors more gracefully#9795
sodonnel merged 1 commit intoapache:HDDS-13323-stsfrom
fmorg-git:HDDS-14681

Conversation

@fmorg-git
Copy link
Contributor

Please describe your PR in detail:

  • When testing Polaris with the feature branch, it was discovered that Polaris is sending StringLike as a condition in the IAM session policy, which previously was being rejected as not supported per the design. This ticket adds that support.
  • Furthermore, throughout the troubleshooting process, it was noticed that certain errors weren't being handled gracefully:
  1. Unsupported Condition operator in IAM session policy was returning 500 Internal Server Error (instead of 501 Not Implemented)
  2. Malformed JSON in IAM session policy was returning 500 Internal Server (instead of 400 Bad Request)
  3. If the STS enabled flag was true, but the OzoneNativeAuthorizer was used instead of Ranger, this returned 500 Internal Server Error (instead of 501 Not Implemented)
  4. When using STS Token for S3 API calls, if the assumed role in Ranger didn't have a requisite permission for the S3 API call, an AccessDenied (403) error was returned, which is fine. However, the OM log had a warn message that the user associated with originalAccessKeyId did not have the permission, which was confusing when the user did actually have that permission and it was the assumed role that did not have the permission.
    These additional 4 issues are also addressed.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14681

How was this patch tested?

unit tests, smoke tests, manual testing and viewing logs

@fmorg-git fmorg-git changed the base branch from master to HDDS-13323-sts February 20, 2026 07:51
Copy link
Contributor

@sodonnel sodonnel left a comment

Choose a reason for hiding this comment

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

LGTM

@sodonnel sodonnel merged commit 6ff9712 into apache:HDDS-13323-sts Feb 20, 2026
83 of 84 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants