-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docker volume for media files #3954
Conversation
Thanks for the PR, maybe we can even simplify even more in the future so unit tests and integration tests can just be run from the dev 'profile'. With the volumes in place and media upload working, should we lookat re-enabling some of the integration tests? Or stop ignoring any (javascript) errors related to those? |
Good idea @valentijnscholten, I will have a look at the integration tests, they should work properly now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cheers Stefan. Let us know if you're ready to merge this one, or if you want it to keep it open a while longer. To avoid a too hasty merge, you can put your PR in draft mode.
The failing unit test is not related to changes in this PR. Damien is working on it. |
Can you take a look @damiencarol ?
|
@madchap, @valentijnscholten, @damiencarol The integration test cases for finding for finding images and file uploads are running now. There is still a thing about http://localhost:8080/static/dojo/img/zoom-in.cur which I don't understand where it comes from. But this will go when the finding images will be removed later this year. From my point of view this PR is ready to be merged. Maybe we want to wait until the failing unit test case is fixed and I have rebased the branch. |
When using Docker Compose, some users experienced problems with permissions of media files, including me. This is caused by the bind mount for the media folder. Docker volumes are the preferred way of managing persistent data in Docker, see https://docs.docker.com/storage/#good-use-cases-for-volumes, and make the application more independent from the infrastructure of the host.
This PR uses a docker volume to store files in the media folder. With this we get an out-of-the box working installation of DefectDojo with Docker Compose. There have been several documentation updates to make users aware they have to of the check the configuration for productive use and adjust it if required.