Skip to content

Remove obsolete DSN identifier#429

Open
Invvard wants to merge 2 commits into
dvd-dev:mainfrom
Invvard:feature/invvard/remove-obsolete-dsn
Open

Remove obsolete DSN identifier#429
Invvard wants to merge 2 commits into
dvd-dev:mainfrom
Invvard:feature/invvard/remove-obsolete-dsn

Conversation

@Invvard

@Invvard Invvard commented Jul 3, 2026

Copy link
Copy Markdown

As part of an effort to remove the now obsolete DSN identifier, we, at Hilo, noticed that the only place where it was mentionned was inside a call to a deprecated endpoint in our backend.

So, here are the changes:

  • Removed the get_gateway() function (and doing so, removing the DSN) from api.py,
  • Cleaned up the devices.pyclass where the aforementionned function was used,
  • Updated some comments to reflect those changes.

@ic-dev21

ic-dev21 commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

As part of an effort to remove the now obsolete DSN identifier, we, at Hilo, noticed that the only place where it was mentionned was inside a call to a deprecated endpoint in our backend.

So, here are the changes:

  • Removed the get_gateway() function (and doing so, removing the DSN) from api.py,
  • Cleaned up the devices.pyclass where the aforementionned function was used,
  • Updated some comments to reflect those changes.

Hey, merci de la contribution! As-tu eu l'occasion de passer ça dans le devcontainer? Sinon pas de stress je vais essayer ça ici. Je ne savais pas que get gateway était mort, ça passe par le devicelist maintenant?

@Invvard

Invvard commented Jul 3, 2026

Copy link
Copy Markdown
Author

Oui, j'ai tout fait via le devcontainer et je me suis assuré que les tests passaient.
Effectivement, ça passe par le websocket désormais. Je crois que le endpoint est toujours présent mais il ne passera pas l'été.
Est-ce que c'est opérationnel du côté de l'intégration? Sinon, on peut se sync pour s'assurer que tout est beau :)

@ic-dev21

ic-dev21 commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

J'ai tout pull dans le devcontainer:

This error originated from a custom integration.

Logger: custom_components.hilo
Source: config_entries.py:798
Integration: Hilo (documentation, issues)
First occurred: 3:49:43 PM (1 occurrence)
Last logged: 3:49:43 PM

Error setting up entry redacted@gmail.com for hilo
Traceback (most recent call last):
  File "/home/vscode/.local/ha-venv/lib/python3.14/site-packages/homeassistant/config_entries.py", line 798, in __async_setup_with_context
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/hilo/custom_components/hilo/__init__.py", line 169, in async_setup_entry
    await hilo.async_init(scan_interval)
  File "/workspaces/hilo/custom_components/hilo/__init__.py", line 758, in async_init
    _async_register_custom_device(
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
        self._hass, self.entry, self.devices.find_device(1)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    )
    ^
  File "/workspaces/hilo/custom_components/hilo/__init__.py", line 120, in _async_register_custom_device
    identifiers={(DOMAIN, device.identifier)},
                          ^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'identifier'

Il manque un petit quelque chose je crois

@ic-dev21

ic-dev21 commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Si je rollback à main sur pyhilo ça fonctionne à nouveau. Je pense qu'il faut aller voir côté device id = 1 (hardcodé comme étant le gateway, qui n'existe plus). Comme il retourne None ça plante.

Comique parce que rien marche dans HA mais le websocket tourne en arrière et je continue à avoir la télémétrie

@ic-dev21

ic-dev21 commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

@Invvard peut-on simplement retirer le DSN pour là, ou le endpoint va mourir de toute manière?

Comme c'est là l'info du gateway ne nous parvient plus du tout, et donc tous les capteurs associés (météo, récompenses, défi, bref, voir screenshot) cessent complètement de fonctionner.
image

Ça va prendre un chemin alternatif pour obtenir l'info du gateway, sans quoi le component Hilo est pas mal mort...

@ic-dev21

ic-dev21 commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

@Invvard ce commit permet d'arracher le call REST + le DSN, mais c'est un quickfix/plaster, c'est fuckall joli.

main...bugfix/ugly-fix-for-dsn

Je vois que l'app mobile permet toujours de voir le DSN et SDI sur mon cell, je suppose que ça va tomber ou que l'info est dispo dans le digital twin?

@Invvard

Invvard commented Jul 4, 2026

Copy link
Copy Markdown
Author

@ic-dev21 Oui, il est toujours disponible pour l'instant.
Pour le moment, on pourrait simplement retirer le DSN. Pour la méthode alternative, je vais voir avec l'équipe qui s'occupe de ce service.
Il n'y a pas non plus d'urgence, on va pas couper ça comme des sauvages ;)

@ic-dev21

ic-dev21 commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

@ic-dev21 Oui, il est toujours disponible pour l'instant. Pour le moment, on pourrait simplement retirer le DSN. Pour la méthode alternative, je vais voir avec l'équipe qui s'occupe de ce service. Il n'y a pas non plus d'urgence, on va pas couper ça comme des sauvages ;)

Non je sais que vous êtes pas des sauvages et si vous saviez à quel point je trouve ça correct de votre part, rien ne vous y oblige. Toujours des heads up, toujours un petit coup de main.

https://discord.gg/MD5ydRJxpc

Passes-nous voir quand tu veux, on va synchroniser ça et faire une job propre :) Merci!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants