Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(mockotlpserver): add http proxy option #608

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

david-luna
Copy link
Member

No description provided.

@trentm
Copy link
Member

trentm commented Feb 14, 2025

Nice. Some notes:

@trentm
Copy link
Member

trentm commented Feb 14, 2025

Also not sure --proxy is the correct term here. Per https://datatracker.ietf.org/doc/html/rfc7230#section-2.3 this feature is more like a tunnel.

(Using --proxy as the optional name could be confusing. Compare, for example, to curl --proxy ... which is about tell curl to use that proxy to send all its outgoing requests to.)

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.

2 participants