Skip to content
This repository has been archived by the owner on Jan 15, 2022. It is now read-only.

Use https instead of http in urls #168

Closed
wants to merge 1 commit into from
Closed

Use https instead of http in urls #168

wants to merge 1 commit into from

Conversation

alanbato
Copy link

@alanbato alanbato commented Jun 6, 2019

This PR changes every instance of an http url to its https version for security reasons.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.001%) to 2.125% when pulling 8a15390 on add_tls_to_urls into e35996b on master.

itsmesatwik
itsmesatwik approved these changes Jun 6, 2019
View reviewed changes
Copy link

@itsmesatwik itsmesatwik left a comment •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Links Work. Build failing tests.

@alanbato alanbato changed the base branch from master to twitter_only June 7, 2019 17:52
@alanbato alanbato changed the base branch from twitter_only to master June 7, 2019 17:54
@alanbato alanbato closed this Jun 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@itsmesatwik itsmesatwik itsmesatwik approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alanbato @coveralls @itsmesatwik