-
Notifications
You must be signed in to change notification settings - Fork 17
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: Add proxy support #11
Conversation
f83b5ce
to
15ece2c
Compare
15ece2c
to
2ee7231
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a small request
Certain implementations may require traffic to pass through a proxy before reaching the outside world (and the LogDNA server). This uses the `https-proxy-agent` package to add proxy capability to the HTTP agent. Semver: minor Ref: LOG-7175
2ee7231
to
b7e68ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
latest with proxy
renaming LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Certain implementations may require traffic to pass through a proxy
before reaching the outside world (and the LogDNA server). This
uses the
https-proxy-agent
package to add proxy capabilityto the HTTP agent.
Semver: minor
Ref: LOG-7175