Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix Multipart Upload for /uploads to work #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wizapp
Copy link

@wizapp wizapp commented Jan 18, 2018

Hi,

For current Strava API, /upload API needs file data to be named "file" in multipart Data. So here is my fix that makes my project working now.

Cheers
Sebastien

@lludo
Copy link
Collaborator

lludo commented Oct 19, 2019

Thank you for this! Is this still the case? Is this in the documentation?

@ehmjaysee
Copy link
Contributor

Uploading activities in the main purpose I was looking into StravaSwift. This looks like a trivial code change so hoping this can get pulled in soon.

@DominicHolmes
Copy link
Contributor

Can confirm that version 1.0.1 doesn't work, but implementing this change lets me successfully upload .gtx files. Thanks so much @wizapp !

See my fork with version 1.0.2 if you want this fix implemented.

@mpclarkson you should consider merging this :)

@lludo
Copy link
Collaborator

lludo commented Apr 15, 2020 via email

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants