Replace spaces with tab for Seccomp check to fix sandboxing

linuxserver / docker-chromium

Web accessible Chromium inside an Debian Container

GNU General Public License v3.0

122 stars 40 forks source link

Replace spaces with tab for Seccomp check to fix sandboxing #12

Closed Lan-Hekary closed 1 year ago

Lan-Hekary commented 1 year ago

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

Description:

Replace spaces with tab for Seccomp check to fix sandboxing

Benefits of this PR and context:

this is to fix the issue in #11

How Has This Been Tested?

Yes, I manually changed the file inside the container and it works as expected with tabs instead of spaces.

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/chromium/427c3596-pkg-427c3596-dev-5052079d150fd2d2e26b78d413846f1a51b921a7-pr-12/index.html https://ci-tests.linuxserver.io/lspipepr/chromium/427c3596-pkg-427c3596-dev-5052079d150fd2d2e26b78d413846f1a51b921a7-pr-12/shellcheck-result.xml	Tag	Passed
amd64-427c3596-pkg-427c3596-dev-5052079d150fd2d2e26b78d413846f1a51b921a7-pr-12	✅
arm64v8-427c3596-pkg-427c3596-dev-5052079d150fd2d2e26b78d413846f1a51b921a7-pr-12	✅

LinuxServer-CI commented 1 year ago

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

thelamer commented 1 year ago

Sorry I missed this, and thanks for the fix. Tested locally and it works waiting for PR build.

github-actions[bot] commented 1 year ago

This pull request is locked due to inactivity