Skip to content

[JENKINS-74098] Remove inline JS script and legacy onClick handler#147

Merged
MarkEWaite merged 4 commits into
jenkinsci:masterfrom
shlomomdahan:JENKINS-74098
Nov 25, 2024
Merged

[JENKINS-74098] Remove inline JS script and legacy onClick handler#147
MarkEWaite merged 4 commits into
jenkinsci:masterfrom
shlomomdahan:JENKINS-74098

Conversation

@shlomomdahan

Copy link
Copy Markdown
Contributor

Jira: JENKINS-74098

What has been done

  1. Remove the inline JS <script> tag as well as its corresponding legacy onChange handler to improve CSP compatibility.

Screenshots

Before
After

How to test

  1. Create a freestyle project
  2. In Post build actions select Plot build data
  3. In the Plot Group enter a name (ex. "test") and use that same plot group name to create 2 or more plots.
  4. Save the configuration, go to the project page and select Plots from the left hand navigation menu
  5. in the "jump to" selector, select a plot and observe the page jump to the location of that plot

Checklist

  • Git commits follow best practices
  • Build passes in Jenkins
  • Appropriate tests or explanation to why this change has no tests
  • Pull Request is marked with appropriate label (see .github/release-drafter.yml)
  • JIRA issue is well described (problem explanation, steps to reproduce, screenshots)
  • For dependency updates: links to external changelogs and, if possible, full diffs

Comment thread src/main/resources/hudson/plugins/plot/PlotReport/jump-to-plot.js Outdated
Comment thread src/main/resources/hudson/plugins/plot/PlotReport/jump-to-plot.js Outdated
@MarkEWaite MarkEWaite added the bug Bug fix label Nov 25, 2024

@MarkEWaite MarkEWaite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@MarkEWaite
MarkEWaite merged commit 3e73a37 into jenkinsci:master Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants