Delete aliyun-java-sdk-sts-internal & Replace AssumeRoleWithServiceId… - #537
Delete aliyun-java-sdk-sts-internal & Replace AssumeRoleWithServiceId…#537TanXiang7o wants to merge 2 commits into
Conversation
…entityRequest with AssumeRoleRequest
RockteMQ-AI
left a comment
There was a problem hiding this comment.
Review: Approved ✅
PR: #537 — Delete aliyun-java-sdk-sts-internal & Replace AssumeRoleWithServiceId
Type: Cleanup (2 files, +4/-11)
Assessment
Removes internal Aliyun SDK dependency and replaces with public API. Reduces code by 7 lines.
Verdict
✅ Correct dependency cleanup. Removes internal-only SDK usage.
🤖 Automated review by oss-sentinel-ai
|
Issue Evaluation Category: This appears to be a tracking issue for replacing Feasibility: feasible (dependency update) Automated evaluation by github-manager-bot |
#536
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice,
it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyleto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.