David Goodwin

Results 134 comments of David Goodwin

If this project isn't dead, it'd be easier for someone to approve/merge this PR if the diff only contained the actual fixes mentioned ... (and not e.g reindentation, as welcome...

@andreamk I'd suggest moving to https://packagist.org/packages/fylax/forceutf8

(the above was just me rebasing this branch on origin/master so i can more easily see what's changed)

Hi - no one has submitted a PR yet for it ... it isn't a priority to me either. I assume it'd require all the templates having absolute URLs (/foo/bar)...

there's always: https://github.com/DavidGoodwin/nmap/blob/master/src/Nmap/XmlOutputParser.php

i'm too stupid to know how to fix it, so it doesn't error on : ```docker run -e POSTFIXADMIN_SETUP_PASSWORD_CLEAR_TEXT=foo\'barbaz ..... ``` gives e.g. ``` + tmp='foo'\''barbaz' + '[' 'foo'\''barbaz' '!='...

Hi - I assume you're referring to PostfixAdmin 3.3.x which looks like : ![image](https://github.com/postfixadmin/postfixadmin/assets/203929/186b1dd0-dc06-437f-ba18-d3285447e3bd) rather than the 'master' version which looks like : ![image](https://github.com/postfixadmin/postfixadmin/assets/203929/f8e1a2b3-e978-4ef6-a48e-d746fb02c3ce) if it's postfixadmin 3.3 then one...

Hi - I think the 'master' codebase is usable on the whole, but I'm fairly sure there are a lot of other changes that have taken place in master, and...

The dependency has been (almost) removed in the master branch of postfixadmin - https://github.com/postfixadmin/postfixadmin/blob/9620056277d09cf7a0e833f60dadc361208d4121/DOCUMENTS/HASHING.md You don't have to use doveadm for your password hashing.

Hi @delgh1 - thanks for raising this. If you're able to create a pull request migrating postfixadmin to bootstrap 5, I'd be happy to merge it :-)