Add option to set overwrite.cli.url for occ install

[kesselb]

Set overwrite.cli.url for occ maintenance:install

Close #6313

[MorrisJobke]

Is really the option needed or can't we just fix the breaking statement? Because I would avoid to put more options in the install command.

Alternative way is to create a file config/NAME.config.php with this as content:

<?php

$CONFIG = [
	'overwrite.cli.url' => 'https://cloud.example.org',
];

This file will then be merge in the config by default and will overwrite the one in config.php.

[kesselb]

Alternative way is to create a file config/NAME.config.php with this as content:

For an automated setup i guess there is no huge difference (appending another option vs. prepare config.php). I took the issue because of https://github.com/nextcloud/server/labels/Hacktoberfest 🙈

[MorrisJobke]

For an automated setup i guess there is no huge difference (appending another option vs. prepare config.php).

It's not about admins, but about us. Because we then need to maintain multiple ways of doing it. 😉

[kesselb]

For an automated setup i guess there is no huge difference (appending another option vs. prepare config.php).

It's not about admins, but about us. Because we then need to maintain multiple ways of doing it. wink

Fine for me 👍

[MorrisJobke]

Fine for me 👍

But the general fix in here is still nice to have. Or did it only fix the issue when the option was provided?

[kesselb]

The issue is only fixed when option provided.

server/lib/private/AppFramework/Http/Request.php

Lines 816 to 822 in c0a283f

	/**
	* Returns the unverified server host from the headers without checking
	* whether it is a trusted domain
	* @return string Server host
	*/
	public function getInsecureServerHost(): string {
	$host = 'localhost';

localhost is coming from this place because there is no way to detect url when using occ.