quick lint issues in main - #7024
Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses gosec lint findings in azdext HTTP helper utilities by suppressing a specific rule at call sites where request URLs are provided dynamically.
Changes:
- Add
//nolint:gosecsuppression for thehttp.Client.Docall insideResilientClient.Do. - Add
//nolint:gosecsuppression for thehttp.Client.Docall inside thestdHTTPDoerpagination adapter.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| cli/azd/pkg/azdext/resilient_http_client.go | Suppresses gosec on rc.httpClient.Do(req) for caller-provided URLs. |
| cli/azd/pkg/azdext/pagination.go | Suppresses gosec on s.client.Do(req) for pager-driven URLs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
No description provided.