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

Upgrade optional dependency commons-email to >= 1.5

    Details

    • Type: Bug Report
    • Status: Closed
    • Priority: L3 - Default
    • Resolution: Fixed
    • Affects Version/s: 7.8.0-alpha4
    • Fix Version/s: 7.9.0, 7.8.3, 7.7.8, 7.6.13, 7.9.0-alpha3
    • Component/s: None
    • Labels:
      None

      Description

      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.

        Activity

        Hide
        roman.smirnov Smirnov Roman added a comment -

        Hi AJ,

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

        Best,
        Roman

        Show
        roman.smirnov Smirnov Roman added a comment - Hi AJ, Sorry for the late response. Which artifact did you scan? Could you please provide more information? Best, Roman
        Hide
        ajbanck 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

        Show
        ajbanck 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
        Hide
        roman.smirnov Smirnov Roman 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

        Show
        roman.smirnov Smirnov Roman 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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development