Skip to content
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

EHRbase vulnerable to CRITICAL Log4j RCE vulnerability #700

Closed
jpwiedekopf opened this issue Dec 10, 2021 · 1 comment · Fixed by #702
Closed

EHRbase vulnerable to CRITICAL Log4j RCE vulnerability #700

jpwiedekopf opened this issue Dec 10, 2021 · 1 comment · Fixed by #702
Assignees
Labels
bug Something isn't working

Comments

@jpwiedekopf
Copy link

Vulnerability

On 2021-12-10, news broke that the popular Log4J library is vulnerable to a very powerful Remote Code Executation bug that leverages the JDNI framework:

EHRbase status

EHRbase has a (transititive) dependency on Log4j2 through the Spring Boot Framework and does not explicitly specify a version of Log4j2 to use. At the time of writing, the docker image on Dockerhub pulls in the latest vulnerable version, 2.14.1 (verified by extracting the ehrbase.jar from the container image):

ehrbase :: ~/ehrbase-jar » find -name "*log*"
./BOOT-INF/lib/log4j-to-slf4j-2.14.1.jar
./BOOT-INF/lib/log4j-core-2.14.1.jar

Upstream Fixes

At the time of writing, an updated version of Log4J has been released that will be delivered to the Spring Boot project in the upcoming release(-s). The upgrade has been committed about 40 minutes ago to the Spring Boot project: spring-projects/spring-boot@95a8e5e

Mitigation

EHRbase should specify an explicit version for Log4j, namely 2.15.0 in its root pom (overriding the version delivered by Spring Boot), and/or should upgrade to the upcoming bugfix-releases of Spring Boot as they become available.

@maximni-vg
Copy link
Contributor

The vulnerable Log4J dependency is updated to a newer, non-vulnerable version in release 0.18.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants