Merge pull request #42 from xmidt-org/lint

ignore linter errors on pw fields

Author: piccione99

internal/consumer/config.go

@@ -58,7 +58,8 @@ type Config struct {
 type SASLConfig struct {
 	Mechanism string
 	Username  string
-	Password  string
+	// #nosec G117 -- field required for configuration, not logged or exposed
+	Password string
 }
 
 // TLSConfig contains TLS encryption configuration.

internal/consumer/consumer_test.go

@@ -201,12 +201,11 @@ func (s *ConsumerTestSuite) TestStart() {
 			} else {
 				s.NoError(err)
 				s.True(s.consumer.IsRunning())
-
-				// Clean up goroutines
-				s.consumer.cancel()
-				s.consumer.wg.Wait()
 			}
 
+			// Clean up goroutines before asserting expectations
+			s.consumer.cancel()
+			s.consumer.wg.Wait()
 			s.mockClient.AssertExpectations(s.T())
 		})
 	}

internal/publisher/config.go

@@ -61,7 +61,8 @@ func (c Config) ToWRPKafkaRoutes() []wrpkafka.TopicRoute {
 type SASLConfig struct {
 	Mechanism string `yaml:"mechanism"`
 	Username  string `yaml:"username"`
-	Password  string `yaml:"password"`
+	// #nosec G117
+	Password string `yaml:"password"`
 }
 
 // TLSConfig represents TLS configuration