aboutsummaryrefslogtreecommitdiff
path: root/cli/do-install.php
diff options
context:
space:
mode:
authorGravatar berumuron <dev@marienfressinaud.fr> 2021-03-26 19:41:33 +0100
committerGravatar GitHub <noreply@github.com> 2021-03-26 19:41:33 +0100
commitcc6c529562ef5751133d97e3fec067c0072f215b (patch)
treee96d2ae665eee11f45607d57ac612a7a7267e314 /cli/do-install.php
parenteeff1a17b0ae13c32560e9d1b59c6e82965f3e6d (diff)
tec: Remove data/do-install.txt (#3555)
* Remove file data/do-install.txt This file was painful during update because we had to remember to delete it each time. It added a security issue by allowing an attacker to reinstall FreshRSS during the update process. The (more powerful) file data/applied_migrations.txt has been introduced in 8619cf6fa to replace do-install.txt. We had to wait for at least one release in order to make sure existing instances of FreshRSS created the migration file. It should be ok now. * Replace i18n install.not_deleted key * Update documentation to update FreshRSS
Diffstat (limited to 'cli/do-install.php')
-rwxr-xr-xcli/do-install.php6
1 files changed, 1 insertions, 5 deletions
diff --git a/cli/do-install.php b/cli/do-install.php
index 4d76b9acb..296b2bd31 100755
--- a/cli/do-install.php
+++ b/cli/do-install.php
@@ -2,7 +2,7 @@
<?php
require(__DIR__ . '/_cli.php');
-if (!file_exists(DATA_PATH . '/do-install.txt')) {
+if (file_exists(DATA_PATH . '/applied_migrations.txt')) {
fail('FreshRSS seems to be already installed!' . "\n" . 'Please use `./cli/reconfigure.php` instead.', EXIT_CODE_ALREADY_EXISTS);
}
@@ -116,8 +116,4 @@ if (!setupMigrations()) {
fail('FreshRSS access right problem while creating migrations version file!');
}
-if (!deleteInstall()) {
- fail('FreshRSS access right problem while deleting install file!');
-}
-
done();