Closed Bug 851192 Opened 11 years ago Closed 11 years ago

Enable geo-location for marketplace

Categories

(Marketplace Graveyard :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED
2013-04-18

People

(Reporter: clouserw, Assigned: wraithan)

Details

Right now we determine what region you want to visit based on your language preferences.  We should do geo-location.

Apparently bug 774746 gave us geo-location, but I don't think those daemons were ever turned on.  CCing JR to give us the current status and we can pick up from there in this bug.
Priority: -- → P1
Assignee: nobody → xwraithanx
Updates:

mkelly has written a geoip server that is more inline with how we'd like to run, as well as being benchmarked: https://github.com/mozilla/geodude

It is currently deployed at http://geo.mozilla.org/country.json to see how it works.

Due to the dataset that we are using, we can't send an arbitrary IP to the server and get back data if the server is public. I'll be forking and adding an endpoint that handles this case. This means we'll need to deploy into our own infrastructure behind the firewall.

I've spoken to cvan to see where in zamboni we need this added, and I'll be coding that up as well as fixing the server. 

Current ETA for the server fork being ready is 2 hours, then I'll be working with ops to get it deployed on a dev server.

Current ETA for the zamboni side is 1-2 hours, it will be behind a waffle so it can be landed and tested on -dev but not affect release this week if we don't have production geoip servers yet.
(In reply to Wraithan from comment #1)
> Due to the dataset that we are using, we can't send an arbitrary IP to the
> server and get back data if the server is public. I'll be forking and adding
> an endpoint that handles this case. This means we'll need to deploy into our
> own infrastructure behind the firewall.

Why would you fork instead of giving a patch to mkelly that has your endpoint behind a flag?
Status: NEW → ASSIGNED
Target Milestone: --- → 2013-04-04
Target Milestone: 2013-04-04 → ---
What's happening with this bug?
Target Milestone: --- → 2013-04-11
Target Milestone: 2013-04-11 → ---
https://github.com/mozilla/zamboni/commit/88656192eb05b560b04718422039df3cbfc6fac9

Code landed. waffle needs to be flipped and a setting added.
(In reply to Wraithan from comment #4)
> https://github.com/mozilla/zamboni/commit/
> 88656192eb05b560b04718422039df3cbfc6fac9
> 
> Code landed. waffle needs to be flipped and a setting added.

what waffle?  what setting?
geoip-geodude is the waffle, the setting comment was more for me, I have a commit that adds them just waiting for r+, then I'll be done with this bug.
https://github.com/mozilla/zamboni/commit/44b250a90c917a4419e99116ba55f6fe801e66e4

Settings in.

`geoip-geodude` is the waffle. I'll ping someone on IRC to enable it where appropriate.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-04-18
You need to log in before you can comment on or make changes to this bug.