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

Start the :ssl application #78

Merged
merged 1 commit into from May 19, 2017

Conversation

2 participants
@randycoulman
Copy link
Contributor

randycoulman commented May 19, 2017

Adds the :ssl application to the list of applications in mix.exs.

When using the default httpc adapter, attempting to GET an https URL would result in the error ** (Tesla.Error) adapter error: :econnrefused.

The :econnrefused error is actually hiding the root cause, which is :ssl_not_started.

It appears that the httpc adapter is not automatically starting the :ssl application.

I tried to find a way to make a test fail, but was unable to. I believe that something else that loads during the test run starts the :ssl application, causing all of the tests to pass.

I did run this version with my application and it now works as I expect.

Fixes #74

Start the :ssl application
Adds the `:ssl` application to the list of applications in `mix.exs`.

When using the default `httpc` adapter, attempting to GET an https URL would result in the error `** (Tesla.Error) adapter error: :econnrefused`.

The `:econnrefused` error is actually hiding the root cause, which is `:ssl_not_started`.

It appears that the httpc adapter is not automatically starting the `:ssl` application.

@teamon teamon merged commit baaa16b into teamon:master May 19, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment