Make sure to assign `production` option in the HERE geocoder constructor

nchaulet / node-geocoder

nodejs geocoding library

http://nchaulet.github.io/node-geocoder/

MIT License

931 stars 214 forks source link

Make sure to assign `production` option in the HERE geocoder constructor #281

Closed dhritzkiv closed 5 years ago

dhritzkiv commented 5 years ago

Addendum to #279 Completely missed this the first time around. When I updated the tests, I believe I was accessing the HereGeocoder constructor directly, thus obscuring this bug.

Sorry that I didn't catch this before a release was pushed.

Perhaps simply passing in extra to the second constructor argument would prevent this mistake in the future?