Repository navigation
Commit d1790ee
committed
[SPARK-54624][UI][3.5] Ensure user name in historypage get escaped
### What changes were proposed in this pull request?
This PR backports #53364 to `branch-3.5`.
This PR aims to escape user name displayed in historypage.
### Why are the changes needed?
Similar to the issue resolved in #52851, user name should also get escaped because arbitrary user name can be set through the env var `SPARK_USER`.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
User name displayed in historypage is escaped even if the name is like `<script>alert('XSS')</script>`
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #53425 from sarutak/fix-username-xss-3.5.
Authored-by: Kousuke Saruta <sarutak@amazon.co.jp>
Signed-off-by: Kousuke Saruta <sarutak@apache.org>1 parent e348c29 commit d1790ee
1 file changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
208 | 208 | | |
209 | 209 | | |
210 | 210 | | |
211 | | - | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
212 | 216 | | |
213 | 217 | | |
214 | 218 | | |
| |||
0 commit comments