-
Notifications
You must be signed in to change notification settings - Fork 79
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
签名时忽略了content-type头 #141
Comments
感谢反馈 因为有一些历史问题 待我们内部评估 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/tencentyun/cos-nodejs-sdk-v5/blob/master/sdk/util.js#L53
https://github.com/tencentyun/cos-js-sdk-v5/blob/master/src/util.js#L53
nodejs和js的sdk计算签名时都没有包含content-type头
gohttps://github.com/tencentyun/cos-go-sdk-v5/blob/master/auth.go#L48
javahttps://github.com/tencentyun/cos-java-sdk-v5/blob/master/src/main/java/com/qcloud/cos/auth/COSSigner.java#L60
phphttps://github.com/tencentyun/cos-php-sdk-v5/blob/master/src/Signature.php#L28
则包含了content-type头
导致nodejs服务端无法实际限制上传类型
The text was updated successfully, but these errors were encountered: