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

fix: update alis for fix alimin program app #1156

Closed
wants to merge 1 commit into from

Conversation

coolnameismy
Copy link

I am from Alibaba Cloud IoT Business Unit. I provide jssdk for developers to connect to the Alibaba Cloud IoT platform (https://github.com/aliyun/alibabacloud-iot-device-sdk). I recently found that the users of Alibaba Applet cannot work normally The reason for using it is that mqttjs uses Alipay APIs that are not standardized (this problem is located with Yunfeng, a classmate of Alipay applet container development), so the correct modification is proposed as follows:

Correct mqttjs error when using Alipay applet mqtt connection
1: my.connectSocket incoming protocol will be intercepted, you need to increase the headers parameter
2: sendSocketMessage data needs to be a base64 string, and isBuffer: 1 is required

issue : #1057

I am from Alibaba Cloud IoT Business Unit. I provide jssdk for developers to connect to the Alibaba Cloud IoT platform (https://github.com/aliyun/alibabacloud-iot-device-sdk). I recently found that the users of Alibaba Applet cannot work normally The reason for using it is that mqttjs uses Alipay APIs that are not standardized (this problem is located with Yunfeng, a classmate of Alipay applet container development), so the correct modification is proposed as follows:

Correct mqttjs error when using Alipay applet mqtt connection
1: my.connectSocket incoming protocol will be intercepted, you need to increase the headers parameter
2: sendSocketMessage data needs to be a base64 string, and isBuffer: 1 is required

issue : mqttjs#1057
@robertsLando
Copy link
Member

It's not clear to me if this can break something for users using ali protocol ?

@robertsLando
Copy link
Member

If you wish to continue with this PR please remember to change destination branch from master to main 🙏🏼

@robertsLando robertsLando changed the title update alis for fix alimin program app fix: update alis for fix alimin program app Jun 30, 2023
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.

None yet

2 participants