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 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

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.

5 participants