Skip to content

[stable10] Return 404 on v2.php when the app is disabled - #1536

Merged
LukasReschke merged 2 commits into
stable10from
backport-1489-ocs-status-code-when-app-is-disabled
Sep 28, 2016
Merged

[stable10] Return 404 on v2.php when the app is disabled#1536
LukasReschke merged 2 commits into
stable10from
backport-1489-ocs-status-code-when-app-is-disabled

Conversation

@nickvergessen

@nickvergessennickvergessen commented Sep 27, 2016

Copy link
Copy Markdown
Member

Backport of #1489

Signed-off-by: Joas Schilling coding@schilljs.com

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessennickvergessen changed the title [stable10]Return 404 on v2.php when the app is disabled[stable10] Return 404 on v2.php when the app is disabledSep 27, 2016
@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Sep 27, 2016
@nickvergessennickvergessen added this to the Nextcloud 10.0.2 milestone Sep 27, 2016
Signed-off-by: Joas Schilling <coding@schilljs.com>

When sending "GET" to "/cloud/capabilities"
Then the OCS status code should be "997"
And the HTTP status code should be "401"

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in master since July: 67ce87d

@nickvergessennickvergessenSep 27, 2016

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and #1538

@MorrisJobke

Copy link
Copy Markdown
Member

Tested and works 👍

@LukasReschke

Copy link
Copy Markdown
Member

LGTM

@LukasReschke
LukasReschke merged commit 766abee into stable10Sep 28, 2016
@LukasReschke
LukasReschke deleted the backport-1489-ocs-status-code-when-app-is-disabled branch September 28, 2016 21:02
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@MorrisJobke@LukasReschke