ftpadmin.sh audit and usability upgrade #2

Merged
Ghost merged 8 commits from shellcheck-ftpadmin into master 2018-11-13 15:37:21 +01:00
First-time contributor

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

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: https://forge.evolix.org/projects/ansible-roles/repository/revisions/388439975b5982ff1e2a9dad8dce81c75ca5c803
Ghost added the
Script
Bug
labels 2018-11-09 23:57:06 +01:00
benpro was assigned by Ghost 2018-11-09 23:57:30 +01:00
Contributor

LGTM (I never touched the FTP part…) more weight decision for @romain.

LGTM (I never touched the FTP part…) more weight decision for @romain.
Author
First-time contributor

Sounds good to me, you can merge your pull request.

Sounds good to me, you can merge your pull request.
Ghost closed this pull request 2018-11-13 15:37:21 +01:00
Ghost deleted branch shellcheck-ftpadmin 2018-11-13 15:37:27 +01:00
Sign in to join this conversation.
No description provided.