-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 bogus http status code for payload too large error #5329
Conversation
Hey @jeanp413 👋 Do we have an issue for this that you can link to or can you please share more info on why this change is good? |
Sure I'll update the title and description. I created this yesterday really quickly to play around with gitpod in my meeting with Anton. |
/werft run 👍 started the job as gitpod-build-jeanp413-bogus-http-status-code-5087.1 |
I changed the reviewer to @akosyakov since he has context. |
/werft run 👍 started the job as gitpod-build-jeanp413-bogus-http-status-code-5087.4 |
It works, but VS Code gives misleading errors: We need to change 100Kb to 1Mb here: https://github.com/gitpod-io/vscode/blob/44f54ed0017685f29b564ee36c79907afd5c090e/src/vs/workbench/contrib/userDataSync/browser/userDataSync.ts#L331-L331 |
412bfee
to
ad3ccce
Compare
/werft run 👍 started the job as gitpod-build-jeanp413-bogus-http-status-code-5087.7 |
ad3ccce
to
3cd7470
Compare
/hold @jeanp413 Could you clean up the history please to remove unnecessary changes in regards to trying different sizes? You can type |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
LGTM label has been added. Git tree hash: ec98285dfcc513c8d083da39adc21be1e9a20f5b
|
/assign @csweichel |
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
LGTM label has been added. Git tree hash: ec98285dfcc513c8d083da39adc21be1e9a20f5b
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: akosyakov, svenefftinge Associated issue: #5087 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes #5087