Skip to content

Catch status code 400 - #2444

Merged
ChristophWurst merged 1 commit into
masterfrom
detect-also-a-400-status
Dec 1, 2016
Merged

Catch status code 400#2444
ChristophWurst merged 1 commit into
masterfrom
detect-also-a-400-status

Conversation

@LukasReschke

Copy link
Copy Markdown
Member

In case the server returns a statuscode 400 we should also gracefully return to the home directory.

Signed-off-by: Lukas Reschke lukas@statuscode.ch

In case the server returns a statuscode 400 we should also gracefully return to the home directory.
Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschkeLukasReschke added the 3. to review Waiting for reviews label Dec 1, 2016
@LukasReschkeLukasReschke added this to the Nextcloud 11.0 milestone Dec 1, 2016
@mention-bot

Copy link
Copy Markdown

@LukasReschke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @butonic, @icewind1991 and @ChristophWurst to be potential reviewers.

@rullzer

Copy link
Copy Markdown
Member

LGTM

@ChristophWurst

Copy link
Copy Markdown
Member

when would that actually happen?

@LukasReschke

LukasReschke commented Dec 1, 2016

Copy link
Copy Markdown
MemberAuthor

@ChristophWurst

Copy link
Copy Markdown
Member

👍 thanks

@ChristophWurst
ChristophWurst merged commit ea45b22 into masterDec 1, 2016
@ChristophWurst
ChristophWurst deleted the detect-also-a-400-status branch December 1, 2016 20:25
@MorrisJobke

Copy link
Copy Markdown
Member

backports: #2445#2446

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@LukasReschke@mention-bot@rullzer@ChristophWurst@MorrisJobke