Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve final text of the installer + setup dir not deleted (r107) #18

Closed
Marc-pi opened this issue Mar 30, 2013 · 5 comments
Closed

Improve final text of the installer + setup dir not deleted (r107) #18

Marc-pi opened this issue Mar 30, 2013 · 5 comments
Assignees

Comments

@Marc-pi
Copy link
Member

Marc-pi commented Mar 30, 2013

Actual text

For security considerations you are strongly recommended to complete the following actions:
Remove the installation folder src from your server.
Set configuration directories and files to readonly:
www/.htaccess
www/boot.php

Support
Visit Pi Engine Development Site in case you need any help.

Improve the text

The install script just completed the following actions :
Remove the installation folder src from your server.
Set configuration directories and files to readonly (chmod 644 on .nix):
www/.htaccess
www/boot.php
Please ensure those have been effectively completed on your environment.

Support
Visit Pi Engine Development Site in case you need any help.

is there any detection message if the script fails to chmod?
marc

and add a detection in admin if the user forgets,

@taiwen
Copy link
Member

taiwen commented Apr 2, 2013

chmod is detected automatically in the installer scripts.

taiwen added a commit that referenced this issue Apr 2, 2013
@taiwen taiwen closed this as completed Apr 2, 2013
@Marc-pi
Copy link
Member Author

Marc-pi commented Apr 27, 2013

hello,
since commit 107, the installer dir www/setup is not deleted anymore. i've tested twice the install. previously it was deleted. after clicking on "visit the website", and log in, i'm back to install.

Additionnaly the line

"Remove the installation folder src from your server"
MUST be change to
"Remove the installation folder directory (www/setup) from your server"

@Marc-pi
Copy link
Member Author

Marc-pi commented May 5, 2013

improved description : ok fixed, thx
redirection issue : ok, fixed, thx
www/setup deletion : not fixed yet, it worked like a charm before, why is it not possible to keep it?
marco

@taiwen
Copy link
Member

taiwen commented May 6, 2013

It requires write permission to delete folders/files, which is more complex and insecure.

@Marc-pi
Copy link
Member Author

Marc-pi commented Feb 9, 2014

replaced by #703

@Marc-pi Marc-pi closed this as completed Feb 9, 2014
taiwen pushed a commit that referenced this issue Aug 30, 2014
Merge pull request #1196 from krisxoofoo/develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants