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

cookieBAR - Timeout for freegeoip #16

Closed
themichael123 opened this issue Sep 9, 2015 · 6 comments
Closed

cookieBAR - Timeout for freegeoip #16

themichael123 opened this issue Sep 9, 2015 · 6 comments

Comments

@themichael123
Copy link

Hi, on every request the following error message is shown in the chrome console:

cookieBAR - Timeout for freegeoip

@ToX82
Copy link
Owner

ToX82 commented Sep 9, 2015

That's a bit of a problem, indeed.
CookieBAR needs freegeoip because the bar should be shown only for the users who are actually subject to the cookie law, the problem is that freegeoip service, while free, is very slow and sometimes it takes 20+ seconds to lookup an ip address.

To avoid such long waiting times, there is a timeout for that request and when the timeout occurs you see it in the console.

I don't really know how to avoid it... apart of just hiding the message itself, but that's not resolving the issue...

@themichael123
Copy link
Author

Thanks for the prompt reply. From my perspective there are several possibilities:

Simple possible solution: Add an API fallback, e.g. http://www.telize.com/geoip

Complex possible solution: Self host the GEO API on AWS Lambda or some other scalable & cheap service.

@ToX82
Copy link
Owner

ToX82 commented Sep 10, 2015

I didn't know telize, looks like a good alternative to freegeoip. I'll do some testing and if it's actually faster I could make the switch...

@ToX82
Copy link
Owner

ToX82 commented Sep 12, 2015

Fixed using telize. Thank you for the suggestion!

@ToX82 ToX82 closed this as completed Sep 12, 2015
@digitall-it
Copy link

//cdn.jsdelivr.net/cookie-bar/1/cookiebar-latest.min.js still seems to refer to 1.5.3.

@digitall-it
Copy link

Seems like the CDN has propagation issues.

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

No branches or pull requests

3 participants