Have Explicit and Distinct `DeepgramHttpClientOptions` and `DeepgramWsClientOptions`
dvonthenen opened this issue · 0 comments
dvonthenen commented
Proposed changes
Currently, all WebSockets and HTTP(s) options are combined into a single ClientOptions class. You run into situations when options like KeepAlive
, which are used for WebSockets, are needed. You can enable KeepAlive
on HTTP(s) clients like ManageClient, but they literally have no function or meaning.
The more the WS and HTTP(s) options grow, the more this object will have options that mean something for one transport type but will have no meaning in the other.
This requires implementing or should I say removing this issue:
#378