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 HTTP(S) Proxy configuration #46

Merged
merged 7 commits into from
Aug 23, 2017
Merged

Conversation

noxdafox
Copy link

@noxdafox noxdafox commented Aug 23, 2017

In case of limited connectivity, this change allows to set HTTP(S) proxy to connect to the service discovery backend.

The following changes have been made since #43 :

  1. Moved to stable branch as target.
  2. Added support for list configuration arguments.
  3. Added tests for utils and config modules.
  4. Renamed config value no_proxy to proxy_exclusions.

Credits to @jjonek.

The as_list function returns the passed value as a list of elements.

Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
Allow the User to specify HTTP proxy to use for connecting to the
service discovery backend.

Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
@noxdafox noxdafox changed the title Stable Add HTTP(S) Proxy configuration Aug 23, 2017
@noxdafox noxdafox force-pushed the stable branch 3 times, most recently from cb3c475 to eb83aab Compare August 23, 2017 21:19
Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
Signed-off-by: Matteo Cafasso <noxdafox@gmail.com>
@michaelklishin michaelklishin merged commit 03a0e46 into rabbitmq:stable Aug 23, 2017
@michaelklishin
Copy link
Member

@noxdafox @jjonek thank you!

@michaelklishin michaelklishin added this to the 0.9.0 milestone Aug 30, 2017
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.

None yet

2 participants