Skip to content

Commit

Permalink
Merge pull request #294 from georchestra/docker-jetty-removes-javax-m…
Browse files Browse the repository at this point in the history
…ail-lib-geor-4233

docker/jetty - removes javax.mail library provided by the docker imag…
  • Loading branch information
pmauduit authored Apr 24, 2024
2 parents ee63e71 + 0050ee2 commit df4dd8f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@
- [ ] PR only involves cherry-picked commits from upstream.
- [ ] PR contains custom code which will soon be available in an upstream release and can be overriden => mention core-geonetwork version if possible.
- [ ] PR contains custom geOrchestra code, which need to be verified during future migrations.
- [ ] I have properly filled the [migration-helper-changelog.md](..%2Fgeorchestra-migration%2Fmigration-helper-changelog.md) file.
- [ ] I have properly filled the [migration-helper-changelog.md](../georchestra-migration/migration-helper-changelog.md) file.

3 changes: 3 additions & 0 deletions web/src/docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ RUN chown jetty:jetty /docker-entrypoint.d
RUN mkdir -p /mnt/geonetwork_datadir && \
chown jetty:jetty /mnt/geonetwork_datadir

# Removes extra javax.mail library provided by Jetty
RUN rm -rf /usr/local/jetty/lib/mail

# Restore jetty user
USER jetty

Expand Down

0 comments on commit df4dd8f

Please sign in to comment.