geoclue-ubuntu-geoip doesn't support proxies

[Jonathan Davies]

geoclue-ubuntu-geoip doesn't seem to support proxy settings, even when they're set in /etc/environment or the System Settings → Network section.

This would be useful for corporate networks where outgoing HTTP is blocked, except for when allowed via a proxy.

[Impact]
System components should respect system proxy settings.

[Test Case]
With non-patched packages, run: kill -9 the geoclue-master and ubuntu-geoip-provider processes while running tcpdump on ports 80 and the port of one's HTTP proxy server.

One will see the connections go straight to port 80 on the IPs above rather than via the proxy.

[Regression Potential]
None.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:ubuntu-geoip at revision 34, scheduled for release in ubuntu-geoip, milestone 1.0.0

[Sebastien Bacher (seb128)]

Thanks for the work there, I've backported the fixes to raring (pre-required for a SRU), I'm sponsoring the precise SRU but declining the quantal one since we try to minimize work on nonLTS series. Feel free to me ping me directly if you really need it upload to the quantal serie though, we can discuss it

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-geoip - 1.0.1-0ubuntu4

---------------
ubuntu-geoip (1.0.1-0ubuntu4) raring; urgency=low

  * Backport upstream commits:
    - Fix ordering of parameters (lp: #1067037)
    - support for proxies (lp: #1130097)
  * debian/control:
    - Build-depends on libsoup-gnome2.4-dev
 -- Sebastien Bacher <email address hidden> Fri, 15 Mar 2013 11:53:53 +0100

[Brian Murray (brian-murray)]

Hello Jonathan, or anyone else affected,

Accepted ubuntu-geoip into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/ubuntu-geoip/0.0.2-0ubuntu6.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Brian Murray (brian-murray)]

Hello Jonathan, or anyone else affected,

Accepted ubuntu-geoip into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/ubuntu-geoip/0.0.2-0ubuntu6.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Jonathan Davies (jpds)]

Patch is missing from debian/patches/series.

[Sebastien Bacher (seb128)]

third try, should be fix this time... sorry about that

@Jonathan: would be nice if you guys did test-build the version you use to submit debdiffs as well... ;-)

[Jonathan Davies (jpds)]

Package verified as OK.

[Scott Kitterman (kitterman)]

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-geoip - 0.0.2-0ubuntu6.3

---------------
ubuntu-geoip (0.0.2-0ubuntu6.3) precise; urgency=low

  * debian/patches/series: list the patch in the serie as well...

ubuntu-geoip (0.0.2-0ubuntu6.2) precise; urgency=low

  * debian/control: build-depends on dh-autoreconf for the previous change

ubuntu-geoip (0.0.2-0ubuntu6.1) precise; urgency=low

  * debian/control: inlcude libsoup-gnome-dev in build dep
  * debian/rules: rebuild configure file
  * debian/patches/proxy.patch: add proxy support (lp: #1130097)
 -- Sebastien Bacher <email address hidden> Thu, 21 Mar 2013 18:19:05 +0100

[Sebastien Bacher (seb128)]

Small status update, raring started receiving a number of segfault with this change: bug #1157232

The issue seems rather in libproxy than ubuntu-geoip and errors.ubuntu.com doesn't suggest it impacts precise