android icon indicating copy to clipboard operation
android copied to clipboard

GetApiUrl check allow insecure connection and refactor IntegrationRepository

Open TimoPtr opened this issue 2 months ago • 1 comments

Summary

This PR makes sure to not provide URL that are not secure to use in the current context based on the allowInsecure flag. It also mutualise the logic in the integration repository to handle errors in the same way everywhere.

Checklist

  • [x] New or updated tests have been added to cover the changes following the testing guidelines.
  • [x] The code follows the project's code style and best_practices.
  • [x] The changes have been thoroughly tested, and edge cases have been considered.
  • [x] Changes are backward compatible whenever feasible. Any breaking changes are documented in the changelog for users and/or in the code for developers depending on the relevance.

Any other notes

Based onto #6131

TimoPtr avatar Dec 10 '25 14:12 TimoPtr

Test Results

104 files  ±0  104 suites  ±0   7m 50s ⏱️ -8s 938 tests +5  938 ✅ +5  0 💤 ±0  0 ❌ ±0  951 runs  +5  951 ✅ +5  0 💤 ±0  0 ❌ ±0 

Results for commit 7c24d9ac. ± Comparison against base commit 1d003d29.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Dec 10 '25 15:12 github-actions[bot]