diff --git a/jsonfxns.go b/jsonfxns.go index a703acd5..ccee3661 100644 --- a/jsonfxns.go +++ b/jsonfxns.go @@ -11,7 +11,7 @@ import ( "io" "io/ioutil" "net/http" - "regexp" + "strings" ) // MarshallAndSend takes the reply structure, marshalls it to json, and @@ -33,13 +33,14 @@ func MarshallAndSend(rawReply Reply, w io.Writer) (string, error) { // than net/rpc/jsonrpc since that one doesn't support http connections and is // therefore useless. func jsonRpcSend(user string, password string, server string, message []byte) (*http.Response, error) { - resp, err := http.Post("http://"+user+":"+password+"@"+server, + credentials := user + ":" + password + resp, err := http.Post("http://"+credentials+"@"+server, "application/json", bytes.NewBuffer(message)) if err != nil { // We do not want to log the username/password in the errors. - re := regexp.MustCompile(`http://\w+:\w+`) - errString := re.ReplaceAllString(fmt.Sprintf("%v", err), "") - err = fmt.Errorf(errString) + replaceStr := ":" + str := strings.Replace(err.Error(), credentials, replaceStr, -1) + err = fmt.Errorf("%v", str) } return resp, err } diff --git a/test_coverage.txt b/test_coverage.txt index 863b0e8d..7203dd6f 100644 --- a/test_coverage.txt +++ b/test_coverage.txt @@ -1,11 +1,11 @@ github.com/conformal/btcjson/jsonapi.go CreateMessage 100.00% (310/310) +github.com/conformal/btcjson/jsonfxns.go jsonRpcSend 100.00% (7/7) github.com/conformal/btcjson/jsonfxns.go MarshallAndSend 100.00% (7/7) github.com/conformal/btcjson/jsonfxns.go GetRaw 100.00% (6/6) -github.com/conformal/btcjson/jsonfxns.go jsonRpcSend 100.00% (6/6) github.com/conformal/btcjson/jsonapi.go jsonWithArgs 100.00% (5/5) github.com/conformal/btcjson/jsonapi.go IsValidIdType 100.00% (3/3) github.com/conformal/btcjson/jsonapi.go RpcCommand 66.67% (18/27) github.com/conformal/btcjson/jsonapi.go readResultCmd 40.00% (20/50) -github.com/conformal/btcjson --------------- 90.58% (375/414) +github.com/conformal/btcjson --------------- 90.60% (376/415)