While debugging a different issue with the OSM Nominatim dev team, it came to their attention that the user agent "drupal.org" is always being sent in the GET. They require something a bit more unique in order to contact the server owner if by chance a request is faulty, or too much info is being requested. I'm guessing in this case the baseurl should be sent rather than "drupal.org".

I set this to bug, since it would be important for sites that perform a lot of requests to avoid interruption.

CommentFileSizeAuthor
#5 i2682527-5.patch1.07 KBattiks
#4 i2682527-4.patch1.08 KBattiks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NWOM created an issue. See original summary.

Pol’s picture

Pol
the/that
Location Brussels
CreditAttribution: Pol as a volunteer commented
Status: Active » Closed (outdated)
NWOM’s picture

NWOM CreditAttribution: NWOM commented
Status: Closed (outdated) » Active

This is still very much relevant and was explicitly requested by the Open Street Maps team.

attiks’s picture

attiks CreditAttribution: attiks at Attiks commented
Status: Active » Needs review
FileSize
i2682527-4.patch1.08 KB

Attached patch changes the default user agent and allows you to override it in settings.php

attiks’s picture

attiks CreditAttribution: attiks at Attiks commented
FileSize
i2682527-5.patch1.07 KB

Ignore #4, new patch

denix’s picture

denix CreditAttribution: denix as a volunteer commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +user agent, +geocoder

It works and I really hope to see this released soon. Nominatim is such a better service!

anrikun’s picture

anrikun CreditAttribution: anrikun commented

See also #3050679: OSM Nominatim does not work

ugintl’s picture

ugintl CreditAttribution: ugintl commented

I think @anrikun's patch covers all nominatim related issues. If it is so then all of them must be merged to avoid confusion.

denix’s picture

denix CreditAttribution: denix as a volunteer commented
Status: Reviewed & tested by the community » Closed (outdated)

I am going to close this as outdated so that we can keep the discussion in #3050679 as suggested by @anrikun