Skip to content

Conversation

MrHinsh
Copy link
Member

@MrHinsh MrHinsh commented Sep 17, 2025

Refactored CreateHyperlink to improve clarity and maintainability.

  • Changed sourceLinkAbsoluteUri declaration to string.
  • Added IsValidHyperlink method to validate hyperlink format and protocol.
  • Introduced GetAbsoluteUri method for reusable URI parsing logic.
  • Enhanced logging to provide detailed warnings for invalid hyperlinks.
  • Improved code readability by modularizing validation and URI handling.

Refactored `CreateHyperlink` to improve clarity and maintainability.
- Changed `sourceLinkAbsoluteUri` declaration to `string`.
- Added `IsValidHyperlink` method to validate hyperlink format and protocol.
- Introduced `GetAbsoluteUri` method for reusable URI parsing logic.
- Enhanced logging to provide detailed warnings for invalid hyperlinks.
- Improved code readability by modularizing validation and URI handling.
@MrHinsh MrHinsh linked an issue Sep 17, 2025 that may be closed by this pull request
@MrHinsh MrHinsh marked this pull request as ready for review September 17, 2025 21:17
@MrHinsh MrHinsh enabled auto-merge September 17, 2025 21:18
@nkdagility-actions-bot
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-river-093197403-2994.westeurope.5.azurestaticapps.net

@MrHinsh MrHinsh merged commit 2e0bf79 into main Sep 17, 2025
15 checks passed
@MrHinsh MrHinsh deleted the 2993-skip-unsupported-or-unsafe-url-protocols-during-work-item-migration branch September 17, 2025 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skip Unsupported or Unsafe URL Protocols During Work Item Migration

1 participant