Support http lookup authentication(oauth2 and token)

[geniusjoe]

support #299

Changes

• Add function HasDataForHttp() and GetHttpHeaders() in AuthenticationDataProvider type to generate http headers authentication properties
• Create a type class called PulsarHttpClient, this class will initialize authentication related members and expose a GetStreamAsync() function
• Replace httpClient.GetStreamAsync() with pulsarHttpClient.GetStreamAsync() in HttpLookupService to enable authentication

[Lanayx]

Async.AwaitTask is not needed here

[Lanayx]

Also, why is it HttpCompletionOption.ResponseHeadersRead?

[geniusjoe]

Async.AwaitTask is not needed here

Fixed

[geniusjoe]

Also, why is it HttpCompletionOption.ResponseHeadersRead?

I changed HttpCompletionOption.ResponseHeadersRead to HttpCompletionOption.ResponseContentRead in my latest commit.

I found that GetStreamAsync() cannot pass custom header, so I searched StackOverflow and refer this answer.

However I think maybe there is no big difference between HttpCompletionOption.ResponseContentRead and HttpCompletionOption.ResponseHeadersRead in small size http response, because these two options are only determining when to read the response:

Indicates if HttpClient operations should be considered completed either as soon as a response is available, or after reading the entire response message including the content.

https://learn.microsoft.com/en-us/dotnet/api/system.net.http.httpcompletionoption?view=net-9.0

[Lanayx]

ResponseContentRead is default, so I think it can just be omitted

[Lanayx]

Actually, I've just realized, that we don't have to have this function return stream. We can make it generic, just like in Java and leverage GetFromJsonAsync built-in method inside

[geniusjoe]

Actually, I've just realized, that we don't have to have this function return stream. We can make it generic, just like in Java and leverage GetFromJsonAsync built-in method inside

@Lanayx
Fixed, please review again. And I change GetStreamAsync() name to Get() to make consistent with Java client.

[Lanayx]

Since this method is not recursive/retriable, we don't need async here, Async.AwaitTask as well, and GetBrokerInner is not needed, this code can be moved to just GetBroker

[geniusjoe]

I removed async and Async.AwaitTask in GetBrokerInner(), and migrate this code to GetBroker() with background{} wrapper.

[Lanayx]

Can you please change seq to array (in the interface as well)

[geniusjoe]

I change HttpLookupService.fs ILookupService and BinaryLookupService GetTopicsUnderNamespace() return type. I think this function only used in PulsarClient.fs.GetTopicsByPattern() so that we don't need to change caller strategy.