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

Add initial support for HTTP proxy basic authentication. #147

Closed
wants to merge 1 commit into from

Conversation

cedric-vincent
Copy link

No description provided.

@tadzik
Copy link
Owner

tadzik commented Apr 13, 2015

I like the patch and I'm going to accept it, but in the current state it introduces MIME::Base64 as a mandatory dependency, and I'd really like to avoid that. Could you change the patch so it loads MIME::Base64 conditionally at runtime, as in: require MIME::Base64 and then if ::("MIME::Base64")? (Writing from memory, the syntax might be slightly different)

@tadzik
Copy link
Owner

tadzik commented Jul 18, 2015

I'm not really keen on growing the in-house HTTP client, so I'm going to close this and open a ticket that'll eventually solve the problem.

@tadzik tadzik closed this Jul 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants