Order matters when passing options to NewClient

[dougwettlaufer]

When passing options to NewClient I noticed that order matters when an http.Client is provided.

This works

knockClient, err := knock.NewClient(
	knock.WithHTTPClient(&http.Client{Transport: GetTracerTransport()}),
	knock.WithAccessToken(knockToken),
)

but if the order is flipped I get a missing token error

knockClient, err := knock.NewClient(
	knock.WithAccessToken(knockToken),
	knock.WithHTTPClient(&http.Client{Transport: cnrtelemetry.GetTracerTransport()}),
)

Seems like this is most likely related to WithHTTPClient stepping directly on the client and wiping out the accessTokenTransport.