Skip to content
This repository has been archived by the owner on Jul 25, 2019. It is now read-only.

Proxy Awareness #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Proxy Awareness #4

wants to merge 1 commit into from

Conversation

hdougie
Copy link

@hdougie hdougie commented May 27, 2014

Currently, the client does not appear to expose any settings for proxy environments. As a result, this means that communication with UserVoice within such an environment does not always work (for example, where outbound internet access is prohibited unless going through the proxy).

RestSharp does have proxy settings available, however the Client class is not currently exposing these. I propose exposing the relevant proxy properties from the RestSharp client class so that proxy connection settings can be configured as required.

Expose the "Proxy" property from RestSharp's "RestClient" class. Allow this to be passed via the "Client" constructor. This will enable proxy configuration for the UserVoice client.
@hdougie
Copy link
Author

hdougie commented May 27, 2014

Is this project still active? Noticed that there has been no activity for some time...

@wlrs
Copy link

wlrs commented May 27, 2014

This project is definitely still alive, but it's small and feature-complete. It sounds like you know exactly what you want to do, if you can submit a pull request we can get it merged. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants