Skip to content

Conversation

@janbuchar
Copy link
Collaborator

@janbuchar janbuchar commented May 17, 2024

Description

TODO

  • copy applicable tests from SDK
  • add the proxy info to context
  • use the configured proxy in HTTP clients

@janbuchar janbuchar requested a review from vdusek May 17, 2024 13:09
@github-actions github-actions bot added this to the 89th sprint - Tooling team milestone May 17, 2024
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label May 17, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Pull Request Tookit has failed!

None of the pull request and linked issue has estimate

@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label May 20, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Pull Request Tookit has failed!

None of the pull request and linked issue has estimate

@janbuchar janbuchar marked this pull request as ready for review May 22, 2024 14:09
@janbuchar janbuchar requested a review from vdusek May 24, 2024 14:25
Copy link
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@janbuchar janbuchar merged commit 5c3753a into master May 24, 2024
@janbuchar janbuchar deleted the proxy-configuration branch May 24, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement ProxyInfo, ProxyConfiguration and tiered proxies

3 participants