refactor: use Uri types instead of strings - #8
Open
deepsource-autofix[bot] wants to merge 1 commit into
Open
Conversation
This PR refactors all URL-related string properties, fields, and methods to use System.Uri, improving type safety and built-in validation across the codebase. - Audit: Consider using `System.URI` instead of `string`s: Previously, many properties and internal fields represented URLs as plain strings, which could lead to invalid formats at runtime and lacked compile‐time enforcement. We converted these members to `System.Uri`, updated getters to return `Uri` instances, modified setters to store the URI via `ToString()`, and adjusted method signatures to accept and return `Uri` where appropriate. > This Autofix was generated by AI. Please review the change before merging.
|
Here's the code health analysis summary for commits Analysis Summary
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR refactors all URL-related string properties, fields, and methods to use System.Uri, improving type safety and built-in validation across the codebase.
System.URIinstead ofstrings: Previously, many properties and internal fields represented URLs as plain strings, which could lead to invalid formats at runtime and lacked compile‐time enforcement. We converted these members toSystem.Uri, updated getters to returnUriinstances, modified setters to store the URI viaToString(), and adjusted method signatures to accept and returnUriwhere appropriate.