Create the default RequestOptions from the client instead of using a required init

[guoye-zhang]

Related discussion in #71

[guoye-zhang]

Should I remove the initializer on the concrete client's options or keep it? It might be confusing whether there is a difference between HTTPRequestOptions() and DefaultHTTPClient.shared.defaultRequestOptions, but it's way less discoverable.

[czechboy0]

Depends on whether you anticipate ever needing to inspect the client's state before constructing a new options value. If not, you could either keep the init, or use something a bit more self-documenting, like:

public static var `default`: Self { .init() }

[czechboy0]

Filed this to continue the nil vs non-nil discussion: #98

[FranzBusch]

NIT: can we move this above the perform method since we normally do properties first then methods.

[guoye-zhang]

I added the empty initializer back. We now have 3 ways to express the same thing:

1. options: HTTPRequestOptions()
2. options: DefaultHTTPClient.shared.defaultRequestOptions
3. options: nil

I would like to discuss if it's actually a good idea for defaultRequestOptions to return differently per client instance, or should RequestOptions just have nil option properties which mean default. E.g. would "minimal" be a configuration passed to the client, then defaultRequestOptions would automatically be minimal?

[FranzBusch]

I added the empty initializer back. We now have 3 ways to express the same thing:

1. options: HTTPRequestOptions()
2. options: DefaultHTTPClient.shared.defaultRequestOptions
3. options: nil

I would like to discuss if it's actually a good idea for defaultRequestOptions to return differently per client instance, or should RequestOptions just have nil option properties which mean default. E.g. would "minimal" be a configuration passed to the client, then defaultRequestOptions would automatically be minimal?

I think we need to think about the developer experience when making the decision here:

// This only works with 2.
func foo(client: some HTTPClient) async throws {
  var options = client.defaultRequestOptions
  options.someSetting = true
}

// .init forces this
func foo<Client: HTTPClient>(client: Client) async throws {
  var options = Client.RequestOptions()
  options.someSetting = true
}

Just for that reason I think we should go with 2.. Now the other question is if we should make the requestOptions parameter on the protocol requirement optional or non-optional. If it is optional we need to define what clients should do. I personally believe it is best to make it non-optional on the protocol and the conveniences can make it optional and state that they call defaultRequestOptions.