jenkinsci / jenkinsci/hetzner-cloud-plugin
[JENKINS-74515] [hetzner-cloud] Migrate legacy checkUrl attribute in cloud/dnation/jenkins/plugins/hetzner/HetznerCloud/config.jelly
- Dominant language
- Java
- Stars
- 28
- Forks
- 11
- PR merge metrics
- No merged PRs in 30d
Description
Problem
```
== Legacy checkUrl
Line: 19
----
checkUrl="'${rootURL}/descriptorByName/cloud.dnation.jenkins.plugins.hetzner.HetznerCloud/checkCloudName?name='+escape(this.value)"
----
```
Solution
https://www.jenkins.io/doc/developer/security/csp/#legacy-javascript-checkurl-validation
---
Originally reported by basil, imported from: [hetzner-cloud] Migrate legacy checkUrl attribute in cloud/dnation/jenkins/plugins/hetzner/HetznerCloud/config.jelly
Raw content of original issue
Problem
== Legacy checkUrl
Line: 19
----
checkUrl="'${rootURL}/descriptorByName/cloud.dnation.jenkins.plugins.hetzner.HetznerCloud/checkCloudName?name='+escape(this.value)"
----Solution
https://www.jenkins.io/doc/developer/security/csp/#legacy-javascript-checkurl-validation
Contributor guide
Research direction
Open cloud/dnation/jenkins/plugins/hetzner/HetznerCloud/config.jelly at line 19 and read the linked Jenkins CSP guidance for legacy checkUrl validation. Migrate this checkUrl usage as described, then verify that the HetznerCloud configuration still performs the cloud-name validation without the legacy attribute.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- devops, security
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100