diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2020-01-04 23:35:42 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2020-01-04 23:35:42 +0100 |
| commit | 2aff347b2e942286292b21e0b20d93ab85220a17 (patch) | |
| tree | a00c587dbaafe36da019dafca00b77357c3f58a8 /cli/do-install.php | |
| parent | acc50df0efef8bd9fa937a5d640314c0e78e9117 (diff) | |
Fix wrong getHeader refactoring (#2749)
* Fix wrong getHeader refactoring
Fix regression introduced by
https://github.com/FreshRSS/FreshRSS/pull/2373
The refactoring required a call to init() even for static functions,
which is most of the time not done.
Removed premature abstraction of `$_SERVER`, which was the root cause of
the bug.
https://github.com/FreshRSS/FreshRSS/issues/2748#issuecomment-569898931
* Refactoring: Move serverIsPublic to Minz_Request
* Add mitigations for wrong configurations
Due to the regression, we have some existing configurations with a bad
base_url
* Forgot one instance
Diffstat (limited to 'cli/do-install.php')
| -rwxr-xr-x | cli/do-install.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/do-install.php b/cli/do-install.php index 16ac833ea..177e98c21 100755 --- a/cli/do-install.php +++ b/cli/do-install.php @@ -53,7 +53,7 @@ foreach ($params as $param) { } } -if ((!empty($config['base_url'])) && server_is_public($config['base_url'])) { +if ((!empty($config['base_url'])) && Minz_Request::serverIsPublic($config['base_url'])) { $config['pubsubhubbub_enabled'] = true; } |