fix: only accept hostname from args only if it's not whitespace

[Cucumberrbob]

User description

related rogerfar/rdt-client#847 (comment)

---

PR Type

Bug fix

---

Description

• Fix hostname validation to reject empty/whitespace strings
• Prevent setting invalid API hostname from constructor arguments

---

Changes walkthrough 📝

	Relevant files
Bug fix	RdNetClient.cs Improve hostname parameter validation                                        RDNET/RdNetClient.cs • Changed null check to String.IsNullOrWhiteSpace() for apiHostname parameter • Prevents setting empty or whitespace-only hostnames in constructor +1/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Logic Issue The condition change may alter existing behavior. Previously, an empty string would be set as the hostname, now it won't be. This could break existing code that relies on empty string being a valid hostname value. if (!String.IsNullOrWhiteSpace(apiHostname)) { _store.ApiHostname = apiHostname; }

[qodo-code-review]

PR Code Suggestions ✨

No code suggestions found for the PR.