Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

更新图片上传api #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

更新图片上传api #21

wants to merge 2 commits into from

Conversation

mrjesen
Copy link
Contributor

@mrjesen mrjesen commented Nov 15, 2020

更新图片上传api-new_api_upload

更新图片上传api-new_api_upload
更新图片上传api的语法错误
@mrjesen mrjesen marked this pull request as ready for review November 15, 2020 23:21
@@ -235,7 +235,7 @@ def uploadPicture(session, image, apis):
'signature':signature
}
data_file = {
'file':(('blob',open(image,'rb')),'image/jpg')
'file':('blob',open(image,'rb'),'image/jpg')
Copy link

Choose a reason for hiding this comment

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

确定不是 image/png 吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

确定

@ghost
Copy link

ghost commented Nov 16, 2020

2020-11-16 13:38:57 {'accessid': 'LTAI7iYVGqI46iFV', 'dir': 'counselor/1019631038321022', 'host': 'https://wecres-dcdn.campusphere.net', 'policy': 'eyJleHBpcmF0aW9uIjoiMjAyMC0xMS0xNlQwNTozOTo1Ny4xMTRaIiwiY29uZGl0aW9ucyI6W1siY29udGVudC1sZW5ndGgtcmFuZ2UiLDAsMTA0ODU3NjAwMF0sWyJzdGFydHMtd2l0aCIsIiRrZXkiLCJjb3Vuc2Vsb3IvMTAxOTYzMTAzODMyMTAyMiJdXX0=', 'signature': 'VudnmM6BydQWwSGFrsov4lYhKyY=', 'fileName': 'counselor/1019631038321022/attachment/b424b970d3ef414bada326daba911a4b'} 2020-11-16 13:38:57 <Response [200]> 2020-11-16 13:38:59 https://wecres.cpdaily.com/counselor/1019631038321022/attachment/b424b970d3ef414bada326daba911a4b.png
经测可用

@ghost
Copy link

ghost commented Nov 21, 2020

整合到我的folk上和readme一起pull了,这是commit地址

@mrjesen
Copy link
Contributor Author

mrjesen commented Nov 22, 2020

整合到我的folk上和readme一起pull了,这是commit地址

我其实是想多个pr,这样的话不太好。。

@ghost
Copy link

ghost commented Nov 22, 2020

那抱歉了2333

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant