Skip to content

[AMORO-3744] Add support for API Token service#3746

Merged
zhoujinsong merged 1 commit intoapache:masterfrom
Jzjsnow:add_support_for_api_token_service
Sep 2, 2025
Merged

[AMORO-3744] Add support for API Token service#3746
zhoujinsong merged 1 commit intoapache:masterfrom
Jzjsnow:add_support_for_api_token_service

Conversation

@Jzjsnow
Copy link
Contributor

@Jzjsnow Jzjsnow commented Aug 21, 2025

Why are the changes needed?

Close #3744 and #1560.

Brief change log

  • Added methods getApiTokens and deleteApiTokenByKey for APITokenManager to manage API Tokens.
  • Added an instance of ApiTokenController and registered related API routes in dashboardServer.
  • Created a ApiTokenController to handle API Token-related requests, including creating, deleting, and querying API Tokens, as well as calculating signatures and encrypting query parameters.
  • Changed the type of applyTime to long and updated the constructor and getter/setter methods accordingly.
  • Added methods getApiTokens and delTokenByKey to the mapper interface.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Copy link
Contributor

@zhoujinsong zhoujinsong 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 for the contribution!

@zhoujinsong zhoujinsong merged commit c237859 into apache:master Sep 2, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-server Ams server module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement]: Add support for API Token service

2 participants