OCR-D / ocrd_all

Master repository which includes most other OCR-D repositories as submodules
MIT License
72 stars 17 forks source link

Clean network logging in dockerimage #429

Open joschrew opened 7 months ago

joschrew commented 7 months ago

Fixes #426

This pr removes the possibly created (while creating the docker image) log-file and folders so that when an docker container with ocrd is created no logfiles and folders exist and thus are created with permissions of the user starting the docker container.

Alternative to https://github.com/OCR-D/core/pull/1214

kba commented 7 months ago

Is this superseded by OCR-D/core#1214?

stweil commented 7 months ago

Is this superseded by OCR-D/core#1214?

I don't think that one is a valid solution because of security concerns.

joschrew commented 7 months ago

Is this superseded by OCR-D/core#1214?

When I thought about the issue I came up with this two (both mentioned pr's) possible solutions. I think both possibly fixes the described problem. The "problem" with this PR is, it only fixes the problem with the ocrd-all-docker-image, and not if similar things happen with other images using ocrd like ocrd-processors.