code cleanup

Author: Romain Dartigues

config.go

@@ -1,6 +1,6 @@
 package clients
 
-// config -
+// Config -
 type Config struct {
 	// Cloud foundry api endoint
 	Endpoint string

rawclient.go

@@ -31,7 +31,7 @@ type RawClientConfig struct {
 	ApiEndpoint       string
 }
 
-// Raw http client has uaa client authentication to make raw request with golang native api.
+// RawClient Raw http client has uaa client authentication to make raw request with golang native api.
 type RawClient struct {
 	connection  cloudcontroller.Connection
 	apiEndpoint string

requestlogger.go

@@ -98,23 +98,23 @@ func NewRequestLogger() *RequestLogger {
 }
 
 func (display *RequestLogger) DisplayBody([]byte) error {
-	log.Print(fmt.Sprintf("%s\n", RedactedValue))
+	log.Printf("%s\n", RedactedValue)
 	return nil
 }
 
 func (display *RequestLogger) DisplayDump(dump string) error {
 	sanitized := display.dumpSanitizer.ReplaceAllString(dump, RedactedValue)
-	log.Print(fmt.Sprintf("%s\n", sanitized))
+	log.Printf("%s\n", sanitized)
 	return nil
 }
 
 func (display *RequestLogger) DisplayHeader(name string, value string) error {
-	log.Print(fmt.Sprintf("%s: %s\n", name, value))
+	log.Printf("%s: %s\n", name, value)
 	return nil
 }
 
 func (display *RequestLogger) DisplayHost(name string) error {
-	log.Print(fmt.Sprintf("host: %s\n", name))
+	log.Printf("host: %s\n", name)
 	return nil
 }
 
@@ -125,27 +125,27 @@ func (display *RequestLogger) DisplayJSONBody(body []byte) error {
 
 	sanitized, err := SanitizeJSON(body)
 	if err != nil {
-		log.Print(fmt.Sprintf("%s\n", string(body)))
+		log.Printf("%s\n", string(body))
 		return nil
 	}
 
-	log.Print(fmt.Sprintf("%s\n", string(sanitized)))
+	log.Printf("%s\n", string(sanitized))
 
 	return nil
 }
 
 func (display *RequestLogger) DisplayMessage(msg string) error {
-	log.Print(fmt.Sprintf("%s\n", msg))
+	log.Printf("%s\n", msg)
 	return nil
 }
 
 func (display *RequestLogger) DisplayRequestHeader(method string, uri string, httpProtocol string) error {
-	log.Print(fmt.Sprintf("%s %s %s\n", method, uri, httpProtocol))
+	log.Printf("%s %s %s\n", method, uri, httpProtocol)
 	return nil
 }
 
 func (display *RequestLogger) DisplayResponseHeader(httpProtocol string, status string) error {
-	log.Print(fmt.Sprintf("%s %s\n", httpProtocol, status))
+	log.Printf("%s %s\n", httpProtocol, status)
 	return nil
 }
 

session.go

@@ -85,32 +85,32 @@ func NewSession(c Config) (s *Session, err error) {
 	return s, nil
 }
 
-// Give access to api cf v3 (complete and always up to date, thanks to cli v7 team)
+// V3 Give access to api cf v3 (complete and always up to date, thanks to cli v7 team)
 func (s *Session) V3() *ccv3.Client {
 	return s.clientV3
 }
 
-// Give access to api uaa (incomplete)
+// UAA Give access to api uaa (incomplete)
 func (s *Session) UAA() *uaa.Client {
 	return s.clientUAA
 }
 
-// Give access to TCP Routing api
+// TCPRouter Give access to TCP Routing api
 func (s *Session) TCPRouter() *router.Client {
 	return s.routerClient
 }
 
-// Give access to networking policy api
+// Networking Give access to networking policy api
 func (s *Session) Networking() *cfnetv1.Client {
 	return s.netClient
 }
 
-// Give an http client which pass authorization header to call api(s) directly
+// Raw Give an http client which pass authorization header to call api(s) directly
 func (s *Session) Raw() *RawClient {
 	return s.rawClient
 }
 
-// Give config store for client which need access token
+// ConfigStore Give config store for client which need access token
 func (s *Session) ConfigStore() *configv3.Config {
 	return s.configStore
 }