Skip to content

Conversation

@ShawkyZ
Copy link
Contributor

@ShawkyZ ShawkyZ commented Nov 6, 2025

Description

Provide description of this PR and changes, if linked Jira ticket doesn't cover it in full.

Checklist

  • Tests added and all succeed
  • Linted
  • README.md updated, if user-facing

🚨After having merged, please update the snyk-ls and CLI go.mod to pull in latest client.

@ShawkyZ ShawkyZ requested review from a team as code owners November 6, 2025 16:36
@snyk-io
Copy link

snyk-io bot commented Nov 6, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions github-actions bot added the fix label Nov 6, 2025
@ShawkyZ ShawkyZ changed the title fix: explain requests shouldnt be encoded fix: explain requests shouldnt be encoded [IDE-1528] Nov 6, 2025
@github-actions github-actions bot added fix and removed fix labels Nov 6, 2025
Copy link
Contributor

@bastiandoetsch bastiandoetsch left a comment

Choose a reason for hiding this comment

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

I think some more test coverage would be good, to ensure the body is checked for if it is encoded.

@ShawkyZ ShawkyZ merged commit 943c98b into main Nov 6, 2025
12 checks passed
@ShawkyZ ShawkyZ deleted the fix/explain-support branch November 6, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants