Skip to content

Allow status.php before install - #1284

Merged
rullzer merged 3 commits into
masterfrom
allow-status.php-before-install
Sep 6, 2016
Merged

Allow status.php before install#1284
rullzer merged 3 commits into
masterfrom
allow-status.php-before-install

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Steps

  1. Remove config.php
  2. curl -v http://nextcloud11.local/status.php

Expected

{"installed":false,"maintenance":false,"version":"9.2.0.3","versionstring":"11.0 alpha","edition":"","productname":"Nextcloud"}

Actual

HTTP/1.1 302 Found

@MorrisJobke@LukasReschke
@longsleep as discussed

@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Sep 6, 2016
@nickvergessennickvergessen added this to the Nextcloud 11.0 milestone Sep 6, 2016
@mention-bot

Copy link
Copy Markdown

@nickvergessen, thanks for your PR! By analyzing the annotation information on this pull request, we identified @MorrisJobke, @mmattel and @LukasReschke to be potential reviewers

@nickvergessen
nickvergessenforce-pushed the allow-status.php-before-install branch from 7f4b264 to 20b636fCompareSeptember 6, 2016 12:14
@rullzer

Copy link
Copy Markdown
Member

I'm fine with this. But what is the reason ;)?

@MorrisJobke

Copy link
Copy Markdown
Member

Tested and works 👍

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

@rullzer spreedbox needs a reliable way to check whether nextcloud is installed

@rullzer

Copy link
Copy Markdown
Member

Fair enough. Ah and of course 👍

@nickvergessennickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Sep 6, 2016
@rullzer
rullzer merged commit 9854705 into masterSep 6, 2016
@rullzer
rullzer deleted the allow-status.php-before-install branch September 6, 2016 13:10
@longsleep

Copy link
Copy Markdown
Member

Nice thanks 👍

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nickvergessen@mention-bot@rullzer@MorrisJobke@longsleep