Setting different HTTP clients rountrippers leads to use the last set roundripper for every client

[pierDipi]

rt1 := // ... create rountripper 
p1 := http.New(http.WithRoundTripper(rt1))

rt2 := // ... create rountripper 
p2 := http.New(http.WithRoundTripper(rt2))

// unexpected:
// p1 has the same roundtripper rt2 as p2

This is caused by the use of the same underlying DefaultClient

sdk-go/v2/protocol/http/protocol.go

Lines 104 to 110 in 4fb49a3

	if p.Client == nil {
	p.Client = http.DefaultClient
	}
	if p.roundTripper != nil {
	p.Client.Transport = p.roundTripper
	}

This can also technically cause data races.

Workaround

Specify a different client for each Protocol using WithClient:

rt1 := // ... create rountripper 
p1 := http.New(http.WithClient(nethttp.Client{Transport: rt1}))

rt2 := // ... create rountripper 
p2 := http.New(http.WithClient(nethttp.Client{Transport: rt2}))

[embano1]

@pierDipi thx for flagging this. Wondering if we can close this issue since you document a workaround?