Skip to content

fix: send Alibaba Cloud SMS GET params in query string not body#13

Open
deepshekhardas wants to merge 2 commits into
mainfrom
fix/6307-alibaba-cloud-adapter
Open

fix: send Alibaba Cloud SMS GET params in query string not body#13
deepshekhardas wants to merge 2 commits into
mainfrom
fix/6307-alibaba-cloud-adapter

Conversation

@deepshekhardas
Copy link
Copy Markdown
Owner

@deepshekhardas deepshekhardas commented May 10, 2026

Summary

Fixes P1 issue identified by cubic:

Issue

Alibaba Cloud SMS signature validation requires query parameters, not request body. Sending signature params in body breaks signature validation and causes API failures.

Fix

  • Moved signature parameters from request body to URL query string
  • GET requests now use proper query string format for Alibaba API

Changes

  • src/Utopia/Messaging/Adapter/SMS/AlibabaCloud.php - Send params as query string instead of body

Identified by cubic (cubic.dev)


Summary by cubic

Send Alibaba Cloud SMS GET parameters in the URL query string instead of the body to match Alibaba’s signing rules. This fixes signature validation failures and restores SMS delivery.

  • Bug Fixes
    • Generate signature over canonicalized, percent-encoded query; include all params in the URL and send GET with no body.
    • Added a basic send test using env vars; treats responses with Code: OK as delivered.

Written for commit bf2c613. Summary will update on new commits.

deepshekhardas added 2 commits March 16, 2026 17:56
Alibaba SMS signature validation requires query parameters, not body.
Moved signature params from request body to URL query string.

Identified by cubic (cubic.dev)
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

You’re at about 99% of the monthly review limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.

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.

1 participant