feat(client): add http support

[zhangyongding]

Complete this issue: Add HTTP support to kmip client

[phsym]

Hi @zhangyongding thanks for this. HTTP client is definitely a good improvement.
I would however prefer avoiding mixing both tcp and http client together in the same client struct.

Maybe a cleaner approach would be to hide transport specific implementation behind an interface that would be used the the Client.conn field, or define an interface over the Client struct and implement it in a dedicated HTTPClient struct. What do you think ?

[zhangyongding]

I think your idea is great.

[ldesauw]

I'm currently working on making the client an interface to allow having specific roundtriper (in your case an http one).
It's not finished yet but I will submit this week.

[zhangyongding]

How is the progress now？

[ldesauw]

#80 added the possibility to precise a custom dialer that will be called.
You can therefore use an http dialer.
An example of implementation is available with the cluster dialer.

Would that be okay for you to define an http dialer using this interface ? Or is there missing pieces ?

[zhangyongding]

Can you provide a verified example?

[ldesauw]

There is one here : https://github.com/ovh/kmip-go/blob/main/kmipclient/dialer_cluster_test.go#L17

[zhangyongding]

https://github.com/ovh/kmip-go/blob/main/kmipclient/dialer_cluster_test.go#L17

Here are all examples using TCP, with no examples using HTTP