Remove db_pass escape logic #140
Conversation
if linuxserver/docker-bookstack#140 works, we should replicate. do not merge until we consider the above PR to be sufficiently tested (but before end of month so i can get a tshirt ;)
|
I am a bot, here are the test results for this PR: |
|
I am a bot, here are the test results for this PR: |
2 similar comments
|
I am a bot, here are the test results for this PR: |
|
I am a bot, here are the test results for this PR: |
We have had some infrequent complaints about DB_PASS not working at times. Unfortunately, the users who complain never provide any relevant detail that allows us to work to resolve the issue. The logic in this is to attempt to escape special characters in the password before feeding it into the database, the users who have the issue erroneously believe that db_password is the right var because of a lack of understanding, however, this does indicate that bypassing our escape logic is a potential solution. I did some testing with various passwords and had no issues by removing our escape logic at all.
If this works, we will want to replicate this PR to mariadb and other containers with a similar var and escape logic.