Skip to content

Comments

test gosec#10

Open
davidaparicio wants to merge 1 commit intomainfrom
conf/volcamp22
Open

test gosec#10
davidaparicio wants to merge 1 commit intomainfrom
conf/volcamp22

Conversation

@davidaparicio
Copy link
Owner

Description

Related Issue(s)

Fixes #

How to test

Release Notes

Documentation

}

if err := srv.ListenAndServe(); err != nil {
if err := http.ListenAndServe(":8080", nil); err != nil {

Check failure

Code scanning / gosec

Use of net/http serve function that has no support for setting timeouts

Use of net/http serve function that has no support for setting timeouts
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

1 similar comment
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

1 participant