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

Replace innerHTML with innerText

XMLWordPrintable

      In https://github.com/jenkinsci/dashboard-view-plugin/blob/b945bace90a5e7b28656f5b254ec5e12d5956681/src/main/resources/hudson/plugins/view/dashboard/stats/StatSlaves/statagents.jelly#L12-L25 there are multiple calls to innerHTML, which are not necessary. A simple innerText is sufficient in this case and will be more safe as innerHTML is vulnerable to XSS. In this particular case it's not (yet) dangerous as the values passed are just integers.

            tgr Tobias Gruetzmacher
            wfollonier Wadeck Follonier
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: