Merge pull request #8 from Financial-Times/rollback-rhys-merge

Revert "Automatically creating CODEOWNERS file [Skip CI]"

README.md

@@ -26,14 +26,22 @@ Initialize logging with `WithLogger` to start logging the outgoing requests.
 
 # Usage
 
-## HTTP Client
-
-For examples how to create new fthttp client refer to `examples_test.go`
-
 ## Transport
+If you need to use the `transport` package separately;
+
+Create a new `*http.Client` which sets a `User-Agent` of `PLATFORM-system-code/version` for all requests:
+
+```
+trans := transport.NewTransport().WithStandardUserAgent("PLATFORM", "system-code")
+client := &http.Client{Transport: trans}
+```
 
-For examples how to use `transport.ExtensibleTransport` refer to the examples in `examples_test.go`
+`NewTransport()` assumes you would like to use the default `http.DefaultRoundTripper`, and the `TransactionIDFromContext` extension.
 
-# Notable Dependencies
+Automatically set the `X-Request-Id` (see the [Transaction ID Utils Go](https://github.com/Financial-Times/transactionid-utils-go) library for more details):
 
-*  [go-logger v2](https://github.com/Financial-Times/go-logger)
+```
+ctx := tidutils.TransactionAwareContext(context.TODO(), "tid_1234")
+req, err := http.NewRequestWithContext(ctx,"GET", uri, nil)
+client.Do(req)
+```

fthttp/client.go

@@ -60,7 +60,6 @@ func WithUserAgent(user string) Option {
 }
 
 // NewClient creates a http.Client object with the provided options
-// If an option fails the produced error would be propagated to the caller of NewClient
 func NewClient(options ...Option) (*http.Client, error) {
 	const defaultClientTimeout = 8 * time.Second
 	client := &http.Client{