Skip to content

fix(report): remove html escaping for shortDescription and fullDescription fields for sarif reports#8344

Merged
DmitriyLewen merged 1 commit intoaquasecurity:mainfrom
DmitriyLewen:fix/sarif-remove-escaping
Feb 4, 2025
Merged

fix(report): remove html escaping for shortDescription and fullDescription fields for sarif reports#8344
DmitriyLewen merged 1 commit intoaquasecurity:mainfrom
DmitriyLewen:fix/sarif-remove-escaping

Conversation

@DmitriyLewen
Copy link
Contributor

Description

See #8342

Related issues

Related PRs

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@DmitriyLewen DmitriyLewen self-assigned this Feb 4, 2025
@DmitriyLewen DmitriyLewen changed the title fix(sarif): remove html escaping for shortDescription and `fullDescription" fields fix(report): remove html escaping for shortDescription and `fullDescription" fields for sarif reports Feb 4, 2025
@DmitriyLewen DmitriyLewen marked this pull request as ready for review February 4, 2025 10:12
@knqyf263 knqyf263 changed the title fix(report): remove html escaping for shortDescription and `fullDescription" fields for sarif reports fix(report): remove html escaping for shortDescription and fullDescription fields for sarif reports Feb 4, 2025
@DmitriyLewen DmitriyLewen added this pull request to the merge queue Feb 4, 2025
Merged via the queue into aquasecurity:main with commit 3eb0b03 Feb 4, 2025
13 of 14 checks passed
@DmitriyLewen DmitriyLewen deleted the fix/sarif-remove-escaping branch February 4, 2025 10:43
@aqua-bot aqua-bot mentioned this pull request Feb 4, 2025
RingoDev referenced this pull request in RingoDev/trivy Feb 26, 2025
dstrelbytskyi referenced this pull request in datarobot/trivy Mar 5, 2025
dstrelbytskyi referenced this pull request in datarobot/trivy Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(sarif): don't escape shortDescription and fullDescription fields

2 participants