The gnome folks would like to also start using our geoip service (at least for now), but their code expects to also find the ip address that requested in the results. They are going into a freeze tomorrow, so they would really prefer this land today.
We have tested in stg and they have run their tests against stg just fine.
I also talked with anaconda folks who currently use this and they don't think there will be any issues with adding a new ip:address result in the output json for them.
Thanks to Xavier for poking at this!
+1s?
kevin -- diff --git a/modules/geoip-city-wsgi/files/geoip-city.wsgi b/modules/geoip-city-wsgi/files/geoip-city.wsgi index 3a00f60..9435820 100755 --- a/modules/geoip-city-wsgi/files/geoip-city.wsgi +++ b/modules/geoip-city-wsgi/files/geoip-city.wsgi @@ -59,6 +59,7 @@ def application(environ, start_response): response.status_code=code return response(environ, start_response)
+ results['ip'] = client_ip results = json.dumps(results) response.headers['Content-Length'] = str(len(results)) response.write(results)
On Mon, Sep 16, 2013 at 10:25:33AM -0600, Kevin Fenzi wrote:
The gnome folks would like to also start using our geoip service (at least for now), but their code expects to also find the ip address that requested in the results. They are going into a freeze tomorrow, so they would really prefer this land today.
We have tested in stg and they have run their tests against stg just fine.
I also talked with anaconda folks who currently use this and they don't think there will be any issues with adding a new ip:address result in the output json for them.
Thanks to Xavier for poking at this!
+1
Pierre
+1
-Toshio On Sep 16, 2013 9:25 AM, "Kevin Fenzi" kevin@scrye.com wrote:
The gnome folks would like to also start using our geoip service (at least for now), but their code expects to also find the ip address that requested in the results. They are going into a freeze tomorrow, so they would really prefer this land today.
We have tested in stg and they have run their tests against stg just fine.
I also talked with anaconda folks who currently use this and they don't think there will be any issues with adding a new ip:address result in the output json for them.
Thanks to Xavier for poking at this!
+1s?
kevin
diff --git a/modules/geoip-city-wsgi/files/geoip-city.wsgi b/modules/geoip-city-wsgi/files/geoip-city.wsgi index 3a00f60..9435820 100755 --- a/modules/geoip-city-wsgi/files/geoip-city.wsgi +++ b/modules/geoip-city-wsgi/files/geoip-city.wsgi @@ -59,6 +59,7 @@ def application(environ, start_response): response.status_code=code return response(environ, start_response)
- results['ip'] = client_ip results = json.dumps(results) response.headers['Content-Length'] = str(len(results)) response.write(results)
infrastructure mailing list infrastructure@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/infrastructure
On Mon, 16 Sep 2013 10:25:33 -0600 Kevin Fenzi kevin@scrye.com wrote:
The gnome folks would like to also start using our geoip service (at least for now), but their code expects to also find the ip address that requested in the results. They are going into a freeze tomorrow, so they would really prefer this land today.
We have tested in stg and they have run their tests against stg just fine.
I also talked with anaconda folks who currently use this and they don't think there will be any issues with adding a new ip:address result in the output json for them.
Thanks to Xavier for poking at this!
+1s?
kevin
diff --git a/modules/geoip-city-wsgi/files/geoip-city.wsgi b/modules/geoip-city-wsgi/files/geoip-city.wsgi index 3a00f60..9435820 100755 --- a/modules/geoip-city-wsgi/files/geoip-city.wsgi +++ b/modules/geoip-city-wsgi/files/geoip-city.wsgi @@ -59,6 +59,7 @@ def application(environ, start_response): response.status_code=code return response(environ, start_response)
- results['ip'] = client_ip results = json.dumps(results) response.headers['Content-Length'] = str(len(results)) response.write(results)
+1 from me
Dennis
And I've moved the geoip-city-wsgi repo from my account on github into the fedora-infra account. Enjoy!
-- Matt Domsch Distinguished Engineer, Director Dell | Software Group
-----Original Message----- From: infrastructure-bounces@lists.fedoraproject.org [mailto:infrastructure-bounces@lists.fedoraproject.org] On Behalf Of Kevin Fenzi
Sent: Monday, September 16, 2013 12:25 PM To: infrastructure@lists.fedoraproject.org Subject: Re: freeze break request: add requesting ip address to geoip
Pushed live.
Thanks everyone.
kevin
On Mon, 16 Sep 2013 13:54:11 -0500 Matt_Domsch@Dell.com wrote:
And I've moved the geoip-city-wsgi repo from my account on github into the fedora-infra account. Enjoy!
Cool. Thanks Matt.
kevin
infrastructure@lists.fedoraproject.org