Skip to content

Commit 19ba564

Browse files
toooniNyholm
authored andcommitted
Add required userAgent (#198)
* Add required userAgent The requirements of Nominatim state that a User-Agent must be submitted (https://operations.osmfoundation.org/policies/nominatim/). It looks like this was already implemented in the provider but not in the config (`ProviderFactory`). So `Geocoder\Provider\Nominatim` does now require a userAgent as third parameter. Solves Issue: #195 * Added mandatory parameter to test config * Made test running again by setting a default user_agent * Made user_agent option as required
1 parent b658141 commit 19ba564

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

ProviderFactory/NominatimFactory.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,20 @@ protected function getProvider(array $config): Provider
2727
{
2828
$httplug = $config['httplug_client'] ?: HttpClientDiscovery::find();
2929

30-
return new Nominatim($httplug, $config['root_url']);
30+
return new Nominatim($httplug, $config['root_url'], $config['user_agent']);
3131
}
3232

3333
protected static function configureOptionResolver(OptionsResolver $resolver)
3434
{
3535
$resolver->setDefaults([
3636
'httplug_client' => null,
3737
'root_url' => 'https://nominatim.openstreetmap.org',
38+
'user_agent' => 'BazingaGeocoderBundle',
3839
]);
3940

4041
$resolver->setAllowedTypes('httplug_client', ['object', 'null']);
4142
$resolver->setAllowedTypes('root_url', ['string']);
43+
$resolver->setAllowedTypes('user_agent', ['string']);
44+
$resolver->setRequired('user_agent');
4245
}
4346
}

Tests/Functional/config/provider/nominatim.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ bazinga_geocoder:
99
factory: Bazinga\GeocoderBundle\ProviderFactory\NominatimFactory
1010
options:
1111
root_url: 'https://nominatim.openstreetmap.org'
12-
12+
user_agent: 'geocoder-php test_suite'

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@
4747
"geocoder-php/tomtom-provider": "^4.0",
4848
"geocoder-php/pickpoint-provider": "^4.0",
4949
"geocoder-php/open-cage-provider": "^4.0",
50-
"geocoder-php/nominatim-provider": "^4.0",
50+
"geocoder-php/nominatim-provider": "^5.0",
5151
"geocoder-php/maxmind-provider": "^4.0",
5252
"geocoder-php/mapzen-provider": "^4.0",
5353
"geocoder-php/mapquest-provider": "^4.0"

0 commit comments

Comments
 (0)