ftpadmin.sh audit and usability upgrade #2
No reviewers
Labels
No labels
Bug
Doc
Feature
Forge
Mode
Cluster
Mode
MultiPHP
Script
Server
Web
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: evolix/evoadmin-web#2
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "shellcheck-ftpadmin"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
I've tested all code paths in the command line and the browser. Make sure you read the individual, per function, commits, as the unified diff is a bit bonkers. But this should make ftpadmin.sh much easier and safer to use. Accompanies the following branch in ansible-roles:
388439975b
LGTM (I never touched the FTP part…) more weight decision for @romain.
Sounds good to me, you can merge your pull request.