Skip to content

Fix G704 adding url validation#1025

Closed
spinillos wants to merge 1 commit intomainfrom
golangci-lint/G704-fix
Closed

Fix G704 adding url validation#1025
spinillos wants to merge 1 commit intomainfrom
golangci-lint/G704-fix

Conversation

@spinillos
Copy link
Copy Markdown
Member

Fixes: #1017

It adds validation for the schema url that we can setup in the configuration. I added "raw.githubusercontent.com" only as allowedHost since is the only one that we are currently use.

@spinillos spinillos self-assigned this Mar 3, 2026
@spinillos spinillos requested a review from a team as a code owner March 3, 2026 11:11
@spinillos spinillos requested review from K-Phoen and konsalex March 3, 2026 11:11
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 3, 2026

Note: the diff show code changes that would be introduced by this PR to the output of the Grafana Foundation SDK codegen pipeline.

See full diff.

🔎 Changes to Grafana Foundation SDK

@K-Phoen
Copy link
Copy Markdown
Member

K-Phoen commented Mar 3, 2026

I think it's one of those occasions when silencing the linting error is the way to go. Doing a request to any URL supplied as input is a feature here, not a bug.

@spinillos spinillos closed this Mar 3, 2026
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.

2 participants