Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-69659

[matrix-project] Un-inlining multiple occurrences

XMLWordPrintable

    • 838.v4d7b_7b_f9b_d4b_

      Level difficult (multiple occurrences, complicated and different setup)
      Skills JavaScript, Jelly

      Culprit 1: LabelAxis/config.jelly#L31-L64 Inline + j:forEach inside

      Culprit 2: MatrixProject/configure-entries.jelly#L52 Inline URL building in the check function (not sure it's not CSP compliant)

      Culprit 3: matrix.jelly#L102-L104 JavaScript method called inlined

          [JENKINS-69659] [matrix-project] Un-inlining multiple occurrences

          Wadeck Follonier created issue -

          Daniel Beck added a comment -

          Inline URL building in the check function (not sure it's not CSP compliant)

          https://www.jenkins.io/doc/developer/security/csp/#legacy-javascript-checkurl-validation

          Daniel Beck added a comment - Inline URL building in the check function ( not sure it's not CSP compliant ) https://www.jenkins.io/doc/developer/security/csp/#legacy-javascript-checkurl-validation
          Wadeck Follonier made changes -
          Assignee Original: Kohsuke Kawaguchi [ kohsuke ]
          Yaroslav Afenkin made changes -
          Assignee New: Yaroslav Afenkin [ yafenkin ]
          Yaroslav Afenkin made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Yaroslav Afenkin made changes -
          Remote Link New: This issue links to "PR (Web Link)" [ 28291 ]
          Yaroslav Afenkin made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Kevin Guerroudj made changes -
          Link New: This issue is duplicated by JENKINS-72134 [ JENKINS-72134 ]
          Kevin Guerroudj made changes -
          Link New: This issue is duplicated by JENKINS-72133 [ JENKINS-72133 ]
          Kevin Guerroudj made changes -
          Link New: This issue is duplicated by JENKINS-72135 [ JENKINS-72135 ]

            yafenkin Yaroslav Afenkin
            wfollonier Wadeck Follonier
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: