Skip to content

feat(services/oss): add support for security token for Aliyun OSS#6511

Merged
Xuanwo merged 1 commit intoapache:mainfrom
cavivie:main
Aug 20, 2025
Merged

feat(services/oss): add support for security token for Aliyun OSS#6511
Xuanwo merged 1 commit intoapache:mainfrom
cavivie:main

Conversation

@cavivie
Copy link
Copy Markdown
Contributor

@cavivie cavivie commented Aug 16, 2025

Which issue does this PR close?

Closes #6507.

Rationale for this change

Security Token Service (STS) allows you to grant time-limited access to data stored on Object Storage Service (OSS) by using temporary access credentials. After the temporary access credentials expire, they can no longer be used to access the data. This helps improve flexibility and timeliness of access control.

What changes are included in this PR?

  • Add the field security_token in OSSConfig.
  • Add the method security_token in OSSBuilder.
  • Improve the documentation for the access_key_id and access_key_secret fields in OssConfig.

Are there any user-facing changes?

  • Add the method security_token in OSSBuilder.

@cavivie cavivie requested a review from Xuanwo as a code owner August 16, 2025 09:46
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Aug 16, 2025
@cavivie
Copy link
Copy Markdown
Contributor Author

cavivie commented Aug 19, 2025

#Anyone feedback here?

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for working on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 20, 2025
@Xuanwo Xuanwo merged commit 110f319 into apache:main Aug 20, 2025
68 of 86 checks passed
@EvanCley
Copy link
Copy Markdown

#Anyone feedback here?

Hi, this commit can backport to 0.48.x?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

new feature: Support custom security token provider for AliyunOSS

3 participants