Cleanup db check so it doesn't spam the console

linuxserver / docker-bookstack

A Docker container for the BookStack documentation wiki

GNU General Public License v3.0

747 stars 108 forks source link

Cleanup db check so it doesn't spam the console #142

Closed thespad closed 1 year ago

thespad commented 1 year ago

[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:

LinuxServer-CI commented 1 year ago

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

nemchik commented 1 year ago

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

thespad commented 1 year ago

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.

github-actions[bot] commented 1 year ago

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.

nemchik commented 1 year ago

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

LinuxServer-CI commented 1 year ago

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