integrations-core icon indicating copy to clipboard operation
integrations-core copied to clipboard

Internal helper class for handling URLs in the base http module

Open iliakur opened this issue 5 months ago • 2 comments

What does this PR do?

Refactoring, no change to user-visible behavior.

Motivation

Avoid parsing the URL multiple times, adding an object that fits our usecase.

Review checklist (to be filled by reviewers)

  • [ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • [ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • [ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

iliakur avatar Jul 24 '25 09:07 iliakur

Codecov Report

:x: Patch coverage is 91.66667% with 3 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 91.06%. Comparing base (e3910c8) to head (667dfee). :warning: Report is 6 commits behind head on master.

Additional details and impacted files
:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jul 24 '25 09:07 codecov[bot]

⚠️ Tests

⚠️ Warnings

🧪 5 Tests failed

test_disk_storage_query from test_queries.py (Datadog)
RetryError[<Future at 0x7fca43f8ead0 state=finished raised SubprocessError>]
test_host_resources_query from test_queries.py (Datadog)
RetryError[<Future at 0x7fca43f8ead0 state=finished raised SubprocessError>]
test_license_audits_query from test_queries.py (Datadog)
RetryError[<Future at 0x7fca43f8ead0 state=finished raised SubprocessError>]
View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 667dfee | Docs | Was this helpful? Give us feedback!