Merge pull request #94 from rollbar/pawel/fix_ip_address

rollbar_test.go

@@ -209,7 +209,7 @@ func TestBuildBodyNoBaseCustom(t *testing.T) {
 func TestErrorRequest(t *testing.T) {
 	r, _ := http.NewRequest("GET", "http://foo.com/somethere?param1=true", nil)
 	r.RemoteAddr = "1.1.1.1:123"
-
+	SetCaptureIp(CaptureIpFull)
 	object := std.requestDetails(r)
 
 	if object["url"] != "http://foo.com/somethere?param1=true" {
@@ -223,6 +223,9 @@ func TestErrorRequest(t *testing.T) {
 	if object["query_string"] != "param1=true" {
 		t.Errorf("wrong id, got %v", object["query_string"])
 	}
+	if object["user_ip"] != "1.1.1.1" {
+		t.Errorf("wrong user_ip, got %v", object["user_ip"])
+	}
 }
 
 func TestRequestForwardedIP(t *testing.T) {

transforms.go

@@ -146,7 +146,9 @@ func remoteIP(req *http.Request) string {
 		ips := strings.Split(forwardedIPs, ", ")
 		return ips[0]
 	}
-	return req.RemoteAddr
+	remoteAddr := req.RemoteAddr
+	spltRemoteAddr := strings.Split(remoteAddr, ":")
+	return spltRemoteAddr[0]
 }
 
 // filterFlatten filters sensitive information like passwords from being sent to Rollbar, and