search

aliyun / aliyun-oss-ruby-sdk

Aliyun OSS SDK for Ruby
http://www.rubydoc.info/gems/aliyun-sdk/
MIT License
140 stars 67 forks source link

修复带 query string 的 object url 签名 #68

Closed tumayun closed 6 years ago

tumayun commented 6 years ago

修复带 query string 的 object url 签名

携带图片处理参数的 object url 签名后无法访问

修复之前：

oss_client.object_url('bar/foo.jpg?x-oss-process=image/resize,m_fill,h_100,w_100', true, 3600)
#=> https://tumayun.oss-cn-beijing.aliyuncs.com/bar/foo.jpg%3Fx-oss-process%3Dimage/resize%2Cm_fill%2Ch_100%2Cw_100?Expires=1525716137&OSSAccessKeyId=OSSAccessKeyId&Signature=XOMywAF0JvLKj1IKuhDX7riwfiQ%3D

修复之后：

oss_client.object_url('bar/foo.jpg?x-oss-process=image/resize,m_fill,h_100,w_100', true, 3600)
#=> https://tumayun.oss-cn-beijing.aliyuncs.com/bar/foo.jpg?x-oss-process=image%2Fresize%2Cm_fill%2Ch_100%2Cw_100&Expires=1525716330&OSSAccessKeyId=OSSAccessKeyId&Signature=%2FXOZrbi3r55Kq0Rkrlf%2Bnzs%2FhJs%3D%3D
coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.009%) to 98.044% when pulling 95e2b896fa80a63dcd6f8501e172fa0361f9ec94 on tumayun:master into 29b8807c4488720eec77b1a2af0bd50aeed76ac5 on aliyun:master.

tumayun commented 6 years ago

@rockuw 请看下，谢谢

tumayun commented 6 years ago

@rockuw 请处理下吧

0x5e commented 6 years ago

Same issue

rockuw commented 6 years ago

抱歉回复得这么晚。

多谢贡献！