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

Use HTTPS where we can, and remove hard-coded port 80 #178

Closed
wants to merge 4 commits into from

Conversation

stephendonner
Copy link
Contributor

Sorry for the delay, and the mess. I've taken another clean shot (hopefully) at this.

@BeyondEvil
Copy link
Contributor

Do all of the providers support HTTPS? Have you tested this? I can test against Testingbot if need be, because I have an account there. @stephendonner

@stephendonner
Copy link
Contributor Author

@BeyondEvil thanks; I could clearly use the help, here. Yes, TestingBot's API supports HTTPS, and is used in their API docs' examples: https://testingbot.com/support/api

The Python client, though, uses primarily HTTP throughout its docs.

@BeyondEvil
Copy link
Contributor

This should now be handled by #190 #191 and #192

@BeyondEvil
Copy link
Contributor

As my previous comment stated, this has now been handled.
You can go head an close this PR if there's nothing else you want to add @stephendonner

Thanks for bringing this to us. 👍

@davehunt davehunt closed this Oct 2, 2018
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

3 participants