Skip to content

feat: support http_proxy_authorization and https_proxy_authorization … #136

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

Merged
merged 2 commits into from
Jul 2, 2025

Conversation

oowl
Copy link
Member

@oowl oowl commented Jul 2, 2025

…options

Copy link

github-actions bot commented Jul 2, 2025

Luacheck Report

4 tests   4 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 30879c7.

♻️ This comment has been updated with latest results.

hackerchai
hackerchai previously approved these changes Jul 2, 2025
@windmgc
Copy link
Member

windmgc commented Jul 2, 2025

LGTM over all. Do we need to define a new configuration var in config.lua?

FYI this may not be a mandatory for us since the AWS CLI only supports basic auth configured in proxy URL https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-proxy.html#cli-configure-proxy-auth

@oowl
Copy link
Member Author

oowl commented Jul 2, 2025

LGTM over all. Do we need to define a new configuration var in config.lua?

FYI this may not be a mandatory for us since the AWS CLI only supports basic auth configured in proxy URL https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-proxy.html#cli-configure-proxy-auth

done, add new http_proxy_authorization and https_proxy_authorization in config.lua

@oowl oowl merged commit 7560e47 into main Jul 2, 2025
9 checks passed
@oowl oowl deleted the https_proxy_auth branch July 2, 2025 08:30
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.

4 participants