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

[feature] SMS notification supports unisms. #3077

Merged
merged 2 commits into from
Feb 15, 2025

Conversation

yunfan24
Copy link
Contributor

@yunfan24 yunfan24 commented Feb 13, 2025

SMS notification supports unisms.

What's changed?

Related issue

  1. Support UniSMS (https://unisms.apistd.com/), including its two authentication methods (simple and HMAC). The API has been tested and verified.
  2. Remove the original TencentCloudApiSignV3.java and extract a more generic encryption algorithm utility class.
  3. Add a new sms-type enum on the frontend and refactor the MessageServerComponent's loadSmsSenderServer() method to ensure compatibility with new SMS service providers without modifying the database-stored configuration structure.

image

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

SMS notification supports unisms.
@zqr10159
Copy link
Member

If you changed application.yml, please don't forget that the project has 5 application.yml files that need to be changed, and they are located in the docker script directory.

@yunfan24
Copy link
Contributor Author

If you changed application.yml, please don't forget that the project has 5 application.yml files that need to be changed, and they are located in the docker script directory.如果您修改了 application.yml,请记住项目中有 5 个 application.yml 文件需要修改,它们位于 docker 脚本目录中。

Thank you for the reminder, the changes have been made.

@zqr10159 zqr10159 merged commit 7d18118 into apache:master Feb 15, 2025
4 checks passed
@yunfan24 yunfan24 deleted the support-unisms branch February 15, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants