core icon indicating copy to clipboard operation
core copied to clipboard

Here Travel Time component uses deprecated protocol

Open jugla opened this issue 4 years ago • 13 comments

The problem

Here Travel Time component uses HERE V3 protocol whereas it is requested to go to V8. This V3 protocol ends at 2021's end. In a deeper way, in manifest.json , the library herepy==2.0.0 is requested. The author of this library proposes the V8 in version 3.5.0 .

What is version of Home Assistant Core has the issue?

2021.4.6

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Core

Integration causing the issue

here_travel_time

Link to integration documentation on our website

https://www.home-assistant.io/integrations/here_travel_time/

Example YAML snippet

- platform: here_travel_time
    name: Home_CommercialCenter_Here_Public_Transportation
    api_key: "xxxxxxxxxxxxxxxxxxx"
    origin_latitude: "48.78096880214132"
    origin_longitude: "2.286129562173117"
    destination_latitude: "48.78521934715194"
    destination_longitude: "2.2160861252742228"
    scan_interval: 1500
    mode: publicTransportTimeTable
    traffic_mode: true

Anything in the logs that might be useful for us?

No response

Additional information

Here the official mail from HERE :

Dear Developer,

You have been identified using deprecated Transit APIs, one or multiple ways:

  • using transit.api.here.com
  • using route.api.here.com with mode=publicTransport
  • using route.api.here.com with mode=publicTransportTimeTable

Above methods of accessing Transit APIs are in maintenance, soon to be deprecated and ultimately going to be decommissioned, latest by end of this year (2021). We kindly ask you to migrate to new unified Transit API V8 to avoid downtime.

Here are few materials and resources our team prepared that can help you:

  • Documentation of the Transit APIs V8 https://developer.here.com/documentation/public-transit/dev_guide/index.html
  • Migration guide transit V3 -> V8 https://developer.here.com/documentation/public-transit/migration_guide/index.html
  • Migration guide routing V7 -> V8 https://developer.here.com/documentation/routing-api/8.21.6/migration_guide/index.html

Best regards, HERE Transit routing team

HERE Europe B.V. Kennedyplein 222-226 5611 ZT Eindhoven, The Netherlands

jugla avatar May 01 '21 06:05 jugla

Hey there @eifinger, mind taking a look at this issue as its been labeled with an integration (here_travel_time) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

Thank you for opening a detailed issue. I already got contacted via Twitter regarding this. I will update the integration and also add a config flow.

eifinger avatar May 01 '21 06:05 eifinger

Thank you !

jugla avatar May 01 '21 07:05 jugla

NB : respect to probot-home-assistant question, I'm the codeowner of worldtidesinfocustom

jugla avatar May 01 '21 07:05 jugla

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jul 30 '21 08:07 github-actions[bot]

I want https://github.com/home-assistant/core/pull/51552 merged before working on this issue to reduce the amount of breaking changes

eifinger avatar Jul 30 '21 08:07 eifinger

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Oct 28 '21 09:10 github-actions[bot]

Still active

eifinger avatar Oct 28 '21 10:10 eifinger

  • [x] Cleanup tests https://github.com/home-assistant/core/pull/60529
  • [x] Use find_coordinates https://github.com/home-assistant/core/pull/59938
  • [x] Use DataUpdateCoordinator https://github.com/home-assistant/core/pull/61398
  • [x] Add a config flow with no breaking changes https://github.com/home-assistant/core/pull/69212
  • [x] Breaking Change: Split up the attributes in different sensors https://github.com/home-assistant/core/pull/72405
  • [x] Make use of SensorDeviceClass https://github.com/home-assistant/core/pull/79159
  • [x] Make use of the v8 HERE api https://github.com/home-assistant/core/pull/80892
  • [x] Make Travel Time entities extend RestoreEntity https://github.com/home-assistant/core/pull/82400
  • [ ] Use strict typing https://github.com/home-assistant/core/pull/83814

eifinger avatar Jan 09 '22 12:01 eifinger

@eifinger Are you still working on this one? Do you have a list of outstanding tasks that you might be happy for someone else to pickup to assist with?

scottgrobinson avatar Oct 11 '22 11:10 scottgrobinson

Yup I'm still working on it and have the PR lined up but am waiting for https://github.com/home-assistant/core/pull/79159

eifinger avatar Oct 11 '22 11:10 eifinger

Thanks - Didn't noticed that linked, much appreciated.

scottgrobinson avatar Oct 11 '22 11:10 scottgrobinson