Issue getlantern#2959 review changes

dolymood · Aug 18, 2015 · 2560e6b · 2560e6b
1 parent fa5d299
commit 2560e6b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/github.com/getlantern/flashlight/logging/logging.go b/src/github.com/getlantern/flashlight/logging/logging.go
@@ -223,7 +223,7 @@ func (w logglyErrorWriter) Write(b []byte) (int, error) {
 		"country":           geolookup.GetCountry(),
 		"timeZone":          w.tz,
 		"version":           w.versionToLoggly,
-		"sessionUserAgents": GetSessionUserAgents(),
+		"sessionUserAgents": getSessionUserAgents(),
 	}
 
 	// extract last 2 (at most) chunks of fullMessage to message, without prefix,

diff --git a/src/github.com/getlantern/flashlight/logging/useragents.go b/src/github.com/getlantern/flashlight/logging/useragents.go
@@ -7,10 +7,8 @@ import (
 	"sync"
 )
 
-type agentsMap map[string]int
-
 var (
-	userAgents  = make(agentsMap)
+	userAgents  = make(map[string]int)
 	agentsMutex = &sync.Mutex{}
 	reg         = regexp.MustCompile("^Go.*package http$")
 )
@@ -34,7 +32,7 @@ func RegisterUserAgent(agent string) {
 }
 
 // getSessionUserAgents returns the
-func GetSessionUserAgents() string {
+func getSessionUserAgents() string {
 	agentsMutex.Lock()
 	defer agentsMutex.Unlock()