Skip to content
This repository was archived by the owner on Feb 28, 2022. It is now read-only.

fix(overpass): use // for the api URL #4

Merged
merged 5 commits into from
Nov 18, 2016
Merged

fix(overpass): use // for the api URL #4

merged 5 commits into from
Nov 18, 2016

Conversation

tomtom44
Copy link
Collaborator

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.768% when pulling 92740ab on tomtom44:patch-1 into 79a21e9 on toutpt:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.768% when pulling 6a4fe48 on tomtom44:patch-1 into 8b37086 on toutpt:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.768% when pulling aff70cb on tomtom44:patch-1 into 8dd888c on toutpt:master.

toutpt
toutpt previously requested changes Nov 17, 2016
View reviewed changes
Copy link
Owner

@toutpt toutpt left a comment •
edited
Loading

Choose a reason for hiding this comment

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

this is bad in both case in fact (http or https).

Could you try with '//' instead of 'https://' and tell me if it's work for you ?

I have missed so many things here !

I have update notifications so now I will answers faster I promise

@tomtom44
Copy link
Collaborator Author

Yes !
'//' is better and works for me :)

@jmfrancois jmfrancois changed the title Update overpass-api.de/api/interpreter URL for HTTPS use fix(overpass): use // for the api URL Nov 18, 2016
@jmfrancois jmfrancois merged commit f4eff38 into toutpt:master Nov 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jmfrancois jmfrancois jmfrancois approved these changes

@toutpt toutpt toutpt left review comments

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

Successfully merging this pull request may close these issues.
4 participants
@tomtom44 @coveralls @toutpt @jmfrancois