Is the nederlandse_spoorwegen integration working?

home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.

https://www.home-assistant.io

Apache License 2.0

72.72k stars 30.45k forks source link

Is the nederlandse_spoorwegen integration working? #24454

Closed ghost closed 5 years ago

ghost commented 5 years ago

Any iemand die home-assistant/homeassistant/components/nederlandse_spoorwegen/ get working?

Couldn't fetch stations, API password correct?: 400 Client Error: Bad Request for url: http://webservices.ns.nl/ns-api-stations-v2

But my password is really correct! Could this plugin or API be old?

andrewsayre commented 5 years ago

Please fill out the issue template. Even though those details may seem unimportant, any tidbit helps.

ghost commented 5 years ago

Hey oke. Here's the error code from "full home assistent log":

2019-06-11 18:52:30 ERROR (SyncWorker_8) [homeassistant.components.nederlandse_spoorwegen.sensor] Couldn't fetch stations, API password correct?: 400 Client Error: Bad Request for url: http://webservices.ns.nl/ns-api-stations-v2

The password is really correct. The code correct also (in de sensor section of configuration.yaml):
platform: nederlandse_spoorwegen email: bla password: bla routes:
- name: Utrecht-Amsterdam from: Ut to: Asd
- name: Amsterdam-Utrecht from: Asd to: Ut

I think there is a problem with ns-api-stations-v2? Could that be?

stokni commented 5 years ago

Exactly the same issue, if you managed to solve it, please let me know.

ghost commented 5 years ago

I did my best to tell the makers here that there is trouble with it. But no response... :-( So still not working...

frenck commented 5 years ago

The API has changed, however, the package Home Assistant uses has not been adjusted to this yet. For more information about this, see: https://github.com/aquatix/ns-api/issues/16

ghost commented 5 years ago

Ah there is a maker! It's sometimes difficult to find out who's the owner to talk to about HomeAssitent modules wich are advertised at https://www.home-assistant.io/components/ Well hope it can be fixed. The trains would be a great addendum to HomeAssistent. :-)

frenck commented 5 years ago

This integration was contributed in the past but currently has no code owner.

Mariusthvdb commented 5 years ago

We’ve all received notice some time ago NS was about to change their api.

In my setup the integration still works fine though ...change will only happen per October, Ive enclosed their mail, (and deleted it again), sorry @Frenck, missed your link above, to the exact same mail.

btw, apologies for the huge image, was sent from my phone, dont know the resize in this editor...

4534232F-E784-405A-8541-859AFCB50FDF

ghost commented 5 years ago

Could it be that if you got that old API in your NS account that it works. And if you have a new NS account, you won't have that old API by default, so the Spoorwegen integration cannot connect to the right API?

Mariusthvdb commented 5 years ago

I wouldn't know tbh. As far as the letter from NS goes, the old Api won't be stopped until October 1st. Personally I haven't setup a new one yet, so you might be right the new one isn't compatible with the current integration ....

just visited the ns-developers website to get a new login, but my current account isn't even recognized.... can't even reset my password. Must be a Friday ;-)

loggin in on http://webservices.ns.nl/ns-api-stations-v2 with my existing credentials works just fine though, I see the full list in the browser

ghost commented 5 years ago

Hmm gonna close this topic to keep things clean. But it's unsolved.