fix(linting): remove unnecessary nolint comments

Author: TheDarthMole

cmd/upswake/serve.go

@@ -114,7 +114,7 @@ func (j *serveJob) sendWakeRequest() {
 		return
 	}
 	req.Header.Set("Content-Type", "application/json")
-	resp, err := j.client.Do(req) //nolint:gosec // G704: This url is only controlled by the user via the config file loaded, which also contains the MAC we are sending anyway
+	resp, err := j.client.Do(req)
 
 	defer func(resp *http.Response) {
 		if resp == nil {

internal/domain/entity/config.go

@@ -74,7 +74,7 @@ type NutServer struct {
 	Host     string          `json:"host"`
 	Port     int             `json:"port"`
 	Username string          `json:"username"`
-	Password string          `json:"password"` //nolint:gosec // G117: TODO investigate secure ways to handle this password, such as using environment variables
+	Password string          `json:"password"`
 	Targets  []*TargetServer `json:"targets"`
 }
 

internal/domain/entity/config_test.go

@@ -151,7 +151,7 @@ func TestNutServer_Validate(t *testing.T) {
 		Host     string
 		Port     int
 		Username string
-		Password string //nolint:gosec // G117: This is a test so we don't care about hardcoding a password here
+		Password string
 		Targets  []*TargetServer
 	}
 	tests := []struct {

internal/infrastructure/config/viper/entities.go

@@ -9,7 +9,7 @@ type NutServer struct {
 	Host     string          `mapstructure:"host"`
 	Port     int             `mapstructure:"port"`
 	Username string          `mapstructure:"username"`
-	Password string          `mapstructure:"password"` //nolint:gosec // G117: TODO investigate secure ways to handle this password, such as using environment variables
+	Password string          `mapstructure:"password"`
 	Targets  []*TargetServer `mapstructure:"targets"`
 }