-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Uploading assets over 20Mb fails. #567
Comments
Hmm, I believe the file size limit is 50MB. https://help.github.com/articles/working-with-large-files I'm not aware if there's a different limit for unauthenticated users. Want to try that first? If it doesn't work, I can dig into it more. |
@haacked this is for the Releases API @distantcam I've not seen anything documented about size limits here: https://developer.github.com/v3/repos/releases/#upload-a-release-asset so i'm now looking behind the scenes for more information... |
@haacked Turns out I was authenticating first. |
Ok, so a colleague reported hitting a file limit of 20Mb. Meanwhile I was having issues with 10Mb. I suspect this is the culprit. https://github.com/octokit/octokit.net/blob/master/Octokit/Http/Request.cs#L14 The connection is timing out before the upload is complete. Yay for Australian internet speeds. |
Right, so it sounds like we need to merge #554 in |
@distantcam can you sit Fiddler in between and see that it's not an exception from the backend you're seeing (i.e. that it actually times out)? |
@shiftkey Yep, not seeing an error from the wire, so it's definitely a timeout. Also the exception I was getting was a Task Cancellation which I'm guessing comes from the HttpClient. |
🆒 so we know where the problem lies at least |
Addressed indirectly in #587 which went out in |
We keep getting a failure when uploading an asset to a release when the asset is over 20Mb. We tested with multiple repos.
Is there a limit in octokit or the GitHub api? Is that limit increased or removed if we authenticate first?
The text was updated successfully, but these errors were encountered: