Fix Update ClientOptions Based on HTTP and WS

[davidvonthenen]

I have been thinking about this a lot and I think separating out HTTP vs WS is probably the better thing to do for ClientOptions. As an example, the KeepAlive (currently) has no meaning for HTTP (yes, I know HTTP does actually support keeping a connection open). Just seems like this is the best way to avoid "overloaded" meanings on this because the object is entirely different.

Other changes:

• Only expose the most common options in each of the constructors as to avoid calling constructors with Constructor(null, null, null, null, something)

Summary by CodeRabbit

• Refactor
  • Enhanced initialization of client options across various tests for consistency, specifically for on-premise configurations.
• Bug Fixes
  • Fixed custom headers setup in HTTP client configuration by adjusting initialization in unit tests.
• Chores
  • Removed unused KeepAlive property and associated logic from client options to streamline HTTP client setup.
• New Features
  • Improved user interaction in the prerecorded file example by adding a prompt to exit the program.

[coderabbitai[bot]]

Walkthrough

The updates focus on enhancing the configuration for the on-premise option across various client tests in the Deepgram SDK, alongside removing the KeepAlive property and its related logic from the DeepgramHttpClientOptions. These changes streamline the initialization of client options and adjust the HTTP client configuration, particularly for on-premise deployments, while simplifying the options model by eliminating less relevant properties.

Changes

Files	Change Summary
.../UnitTests/ClientTests/*.cs	Set OnPrem to true using object initializer in client options.
.../UnitTests/HttpExtensionsTests/HttpClientExtensionTests.cs	Updated _clientOptions initialization by adjusting custom headers setup.
Deepgram/Abstractions/AbstractRestClient.cs	Removed logging of KeepAlive option.
Deepgram/Models/Authenticate/v1/DeepgramHttpClientOptions.cs	Removed KeepAlive property and updated constructor signature.
Deepgram/Models/Authenticate/v1/DeepgramWsClientOptions.cs	Updated constructor signature and modified sections.
examples/prerecorded/file/Program.cs	Added prompt for user to press any key to exit after displaying JSON response.

Possibly related issues

• deepgram/deepgram-dotnet-sdk#234: The refactoring mentioned in the PR could contribute to the thread safety improvements and customization of HTTP parameters for the query string as outlined in the issue. The removal of KeepAlive and adjustments in client option initializations align with the broader goal of refining HTTP/REST client configurations.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share

- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)

---

Tips

### Chat There are 3 ways to chat with [CodeRabbit](https://coderabbit.ai): - Review comments: Directly reply to a review comment made by CodeRabbit. Example: - `I pushed a fix in commit .` - `Generate unit testing code for this file.` - `Open a follow-up GitHub issue for this discussion.` - Files and specific lines of code (under the "Files changed" tab): Tag `@coderabbitai` in a new review comment at the desired location with your query. Examples: - `@coderabbitai generate unit testing code for this file.` - `@coderabbitai modularize this function.` - PR comments: Tag `@coderabbitai` in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples: - `@coderabbitai generate interesting stats about this repository and render them as a table.` - `@coderabbitai show all the console.log statements in this repository.` - `@coderabbitai read src/utils.ts and generate unit testing code.` - `@coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.` Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. ### CodeRabbit Commands (invoked as PR comments) - `@coderabbitai pause` to pause the reviews on a PR. - `@coderabbitai resume` to resume the paused reviews. - `@coderabbitai review` to trigger a review. This is useful when automatic reviews are disabled for the repository. - `@coderabbitai resolve` resolve all the CodeRabbit review comments. - `@coderabbitai help` to get help. Additionally, you can add `@coderabbitai ignore` anywhere in the PR description to prevent this PR from being reviewed. ### CodeRabbit Configration File (`.coderabbit.yaml`) - You can programmatically configure CodeRabbit by adding a `.coderabbit.yaml` file to the root of your repository. - Please see the [configuration documentation](https://docs.coderabbit.ai/guides/configure-coderabbit) for more information. - If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: `# yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json` ### Documentation and Community - Visit our [Documentation](https://coderabbit.ai/docs) for detailed information on how to use CodeRabbit. - Join our [Discord Community](https://discord.com/invite/GsXnASn26c) to get help, request features, and share feedback. - Follow us on [X/Twitter](https://twitter.com/coderabbitai) for updates and announcements.