On Mon, 16 Sep 2013 10:25:33 -0600 Kevin Fenzi <kevin@xxxxxxxxx> 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
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/infrastructure
