docker-bookstack icon indicating copy to clipboard operation
docker-bookstack copied to clipboard

Cleanup db check so it doesn't spam the console

Open thespad opened this issue 3 years ago • 3 comments

linuxserver.io


  • [x] I have read the contributing guideline and understand that I have made the correct modifications

Description:

Stops the /etc/cont-init.d/50-config: line 102: warning: command substitution: ignored null byte in input spam during startup.

Also remove curl and netcat as they're on the base.

Benefits of this PR and context:

How Has This Been Tested?

Source / References:

thespad avatar Oct 17 '22 15:10 thespad

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/bookstack/v22.09.1-pkg-e64e3215-pr-142/index.html https://ci-tests.linuxserver.io/lspipepr/bookstack/v22.09.1-pkg-e64e3215-pr-142/shellcheck-result.xml

LinuxServer-CI avatar Oct 17 '22 15:10 LinuxServer-CI

Let's wait for https://github.com/linuxserver/docker-bookstack/pull/136 which touches some of the same code.

nemchik avatar Oct 17 '22 16:10 nemchik

Let's wait for #136 which touches some of the same code.

Forgot about that PR, just had to do something similar for another container startup, so I thought I'd apply it over here too.

thespad avatar Oct 17 '22 16:10 thespad

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Nov 17 '22 03:11 github-actions[bot]

This can now be rebased since https://github.com/linuxserver/docker-bookstack/pull/136 is merged

nemchik avatar Dec 22 '22 22:12 nemchik

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/bookstack/v22.11.1-pkg-9a84e426-pr-142/index.html https://ci-tests.linuxserver.io/lspipepr/bookstack/v22.11.1-pkg-9a84e426-pr-142/shellcheck-result.xml

LinuxServer-CI avatar Dec 23 '22 10:12 LinuxServer-CI