Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-8277

Upgrade optional dependency commons-email to >= 1.5

XMLWordPrintable

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 7.9.0, 7.8.3, 7.7.8, 7.6.13, 7.9.0-alpha3
    • 7.8.0-alpha4
    • None
    • None

      OWASP scan of Camunda shows a known vulnerability in the commons-email version used in Camunda

      http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2017-9801
      When a call-site passes a subject for an email that contains line-breaks in Apache Commons Email 1.0 through 1.4, the caller can add arbitrary SMTP headers.

        This is the controller panel for Smart Panels app

            [CAM-8277] Upgrade optional dependency commons-email to >= 1.5

            Roman Smirnov added a comment -

            Hi AJ,

            Sorry for the late response. Which artifact did you scan? Could you please provide more information?

            Best,
            Roman

            Roman Smirnov added a comment - Hi AJ, Sorry for the late response. Which artifact did you scan? Could you please provide more information? Best, Roman

            AJ added a comment - - edited

            This is in Camunda-engine:

            +- org.camunda.bpm:camunda-engine-spring:jar:7.8.0:compile
|  +- org.camunda.bpm:camunda-engine:jar:7.8.0:compile
|  |  +- org.apache.commons:commons-email:jar:1.2:compile

            The commons-email version is defined in comunda-bpm-platform/camunda-parent Currently it is still at version 1.2
            https://github.com/camunda/camunda-bpm-platform/blob/master/parent/pom.xml#L189

            AJ added a comment - - edited This is in Camunda-engine: +- org.camunda.bpm:camunda-engine-spring:jar:7.8.0:compile | +- org.camunda.bpm:camunda-engine:jar:7.8.0:compile | | +- org.apache.commons:commons-email:jar:1.2:compile The commons-email version is defined in comunda-bpm-platform/camunda-parent Currently it is still at version 1.2 https://github.com/camunda/camunda-bpm-platform/blob/master/parent/pom.xml#L189

            Roman Smirnov added a comment -

            Hi AJ,

            Thanks for sharing more information.

            To let you know, commons-email is an optional depedency, and it is only needed when you are using a mail task. So, in case you are embedding the engine in you application, you can easily exclude this dependency. However, the pre-packaged distrubtions don't include that dependency.

            Anyway, we are going to check, if we can upgrade commons-email to > 1.5.

            Best,
            Roman

            Roman Smirnov added a comment - Hi AJ, Thanks for sharing more information. To let you know, commons-email is an optional depedency, and it is only needed when you are using a mail task. So, in case you are embedding the engine in you application, you can easily exclude this dependency. However, the pre-packaged distrubtions don't include that dependency. Anyway, we are going to check, if we can upgrade commons-email to > 1.5. Best, Roman

              roman.smirnov Roman Smirnov
              ajbanck AJ
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: