search

scaarup / aula

Fetches information from Aula about your children
GNU General Public License v3.0
63 stars 17 forks source link

Får ingen data fra Aula #191

Closed dennisroenn77 closed 1 week ago

dennisroenn77 commented 3 weeks ago

Hej,

Tak for en fantastisk integration. Den hjælper SÅ meget i hverdagen.

Igår ca. kl. 21 mistede jeg alle sensorer mv. Jeg her gentagene gange forsøgt at fjerne både Aula fra HACS og selve integrationen, og installere igen. Jeg får nedenstående error log.

(Jeg har tjekket at mit Unilogin virker på Aula's hjemme side, så det er ikke et forkert pw jeg taster :-) Sidste ting: Det er mit eget Unilogin jeg bruger ikke børnenes. Jeg formoder det er rigtigt.

Please answer the following

REQUIRED! Provide debug log from the integration

Jogge commented 3 weeks ago

Det er ugeplaner som fejler igen...

Indtil det er løst kan du slå ugeplaner fra under opsætningskonfigurationen, hvor du skal undlade at slå følgende til:

image

Husk at slet din eksisterende opsætning først, så du ikke får duplicates:

image

Oceanfarm commented 3 weeks ago

Ikke her. Den efter at have slettet enheden, finder den intet efter tilføjelsen på ny.

image
scaarup commented 3 weeks ago

Alle jer med problemer her, bruger I Meebook ('0004': 'Ugeplan forældre',) ? Og er der problemer hos dem - får I data i Aula app'en uden problemer?

Oceanfarm commented 3 weeks ago

Har problemer både med ugeplan valgt til og fra.

Får data i Aula app uden problemer.

scaarup commented 3 weeks ago

Har problemer både med ugeplan valgt til og fra.

Får data i Aula app uden problemer.

Der skal lidt debug log til, hvis vi skal kunne konstatere, at det er det samme problem som @dennisroenn77 har rapporteret her.

thomaspalmdk commented 3 weeks ago

Jeg har samme problem; integrationen er holdt helt op med at virke, og det er ikke bare det sædvanlige password.

dennisroenn77 commented 3 weeks ago

Hej Søren, Jeg kan bekæfte at løsningen med at undlade fluebenet i "Add ugeplaner as sensor attributes" virker. Efter en omgang slet og installere integrationen på ny, virker det igen. Dvs. jeg får kalenderen med skemaet oprettet, men givet, ingen ugeplaner (i sensorerne).

Et forsøg på lidt info som måske kan hjælpe til løsningen på problemet: Det ser ud til at i Aula app'en, at ugeplaner fungerer korrekt - både hvis man i menuen trykker "Ugeplan forældre" eller Meebook overblik. Jeg har indsat et par sceenshots fra appen herunder.

Måske der er andre som kan sende deres debug log hvis i oplever samme fejl - altså at der ikke kan hentes data fra Ugeplan? Det vil sikkert også hjælpe.

Endnu engang, tak for en god integration :-)

Screenshot_20241031_213052-EDIT-EDIT Screenshot_20241031_213042-EDIT Screenshot_20241031_213306-EDIT

poulpreben commented 3 weeks ago

Jeg har haft debug slået til siden i går, og her er en fejl som måske kan hjælpe os videre:

2024-10-31 15:00:32.587 INFO (MainThread) [custom_components.aula] 
                _
     /\        | |
    /  \  _   _| | __ _
   / /\ \| | | | |/ _` |
  / ____ \ |_| | | (_| |
 /_/    \_\__,_|_|\__,_|
Aula integration, version: 0.1.42
This is a custom integration
If you have any issues with this you need to open an issue here:
https://github.com/scaarup/aula/issues
-------------------------------------------------------------------

2024-10-31 15:00:32.596 DEBUG (SyncWorker_5) [custom_components.aula.client] is_logged_in? False
2024-10-31 15:00:32.596 DEBUG (SyncWorker_5) [custom_components.aula.client] Logging in
2024-10-31 15:00:34.897 DEBUG (SyncWorker_5) [custom_components.aula.client] Trying API at https://www.aula.dk/api/v19
2024-10-31 15:00:34.939 DEBUG (SyncWorker_5) [custom_components.aula.client] API was expected at https://www.aula.dk/api/v19 but responded with HTTP 410. The integration will automatically try a newer version and everything may work fine.
2024-10-31 15:00:34.939 DEBUG (SyncWorker_5) [custom_components.aula.client] Trying API at https://www.aula.dk/api/v20
2024-10-31 15:00:35.146 DEBUG (SyncWorker_5) [custom_components.aula.client] Found API on https://www.aula.dk/api/v20
2024-10-31 15:00:36.980 DEBUG (SyncWorker_5) [custom_components.aula.client] LOGIN: True
2024-10-31 15:00:36.980 DEBUG (SyncWorker_5) [custom_components.aula.client] Config - schoolschedule: True, config - ugeplaner: True
2024-10-31 15:00:36.980 DEBUG (SyncWorker_5) [custom_components.aula.client] Child ids and names: {nnn: 'xxx'}
2024-10-31 15:00:36.980 DEBUG (SyncWorker_5) [custom_components.aula.client] Child ids and institution names: {nnn: 'Hanssted Skole'}
2024-10-31 15:00:36.980 DEBUG (SyncWorker_5) [custom_components.aula.client] Institution codes: ['nnn']
2024-10-31 15:00:39.392 DEBUG (SyncWorker_5) [custom_components.aula.client] Child ids and presence data status: {'nnn': 1}
2024-10-31 15:00:42.596 WARNING (MainThread) [homeassistant.components.sensor] Setup of sensor platform aula is taking over 10 seconds.
2024-10-31 15:01:00.882 ERROR (MainThread) [custom_components.aula.sensor] Unexpected error fetching sensor data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 354, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 271, in _async_update_data
    return await self.update_method()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/aula/sensor.py", line 60, in async_update_data
    await hass.async_add_executor_job(client.update_data)
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/aula/client.py", line 762, in update_data
    ugeplan(thisweek, "this")
  File "/config/custom_components/aula/client.py", line 743, in ugeplan
    ugep = ugep + task["author"] + "<br><br>"
                  ~~~~^^^^^^^^^^
KeyError: 'author'
2024-10-31 15:01:12.829 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up aula platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform
    await asyncio.shield(awaitable)
  File "/config/custom_components/aula/sensor.py", line 75, in async_setup_entry
    await hass.async_add_executor_job(client.update_data)
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/aula/client.py", line 762, in update_data
    ugeplan(thisweek, "this")
  File "/config/custom_components/aula/client.py", line 743, in ugeplan
    ugep = ugep + task["author"] + "<br><br>"
                  ~~~~^^^^^^^^^^
KeyError: 'author'
2024-10-31 15:01:12.830 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration aula with title: Aula and entry_id: 01JBHB6A9D7FYNPRDG4217FJCC, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.
Stack (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
    sys.exit(main())
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/usr/src/homeassistant/homeassistant/runner.py", line 189, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2223, in async_forward_entry_setup
    _report_non_awaited_platform_forwards(entry, "async_forward_entry_setup")
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1192, in _report_non_awaited_platform_forwards
    report(
  File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 155, in report
    _LOGGER.warning(msg, stack_info=True)

Jeg rettede følgende linje til, og nu virker det igen:

https://github.com/scaarup/aula/blob/07da541e2326bb9e4684812c45cd55264eaa74b5/custom_components/aula/client.py#L759

author = task.get('author', "")
ugep = ugep + author + "<br><br>"
Oceanfarm commented 3 weeks ago

Jeg er måske ikke helt med. Du skriver at du rettede linjen (ental) til, men har tilføjet to linjer nedenfor? Så du har skiftet linje 759 ud med de to linjer indført? I så fald, virker det stadig ikke her...

Oceanfarm commented 3 weeks ago

2024-11-01 16:13:59.635 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up aula platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform await asyncio.shield(awaitable) File "/config/custom_components/aula/sensor.py", line 75, in async_setup_entry await hass.async_add_executor_job(client.update_data) File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/aula/client.py", line 778, in update_data nextweek = now.strftime("%Y-W%W") ^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/aula/client.py", line 759, in ugeplan author = task.get('author', "") ^^^^^^ NameError: name 'author' is not defined 2024-11-01 16:13:59.649 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration aula with title: Aula and entry_id: 01JBKSASVP0TRYKR45BEPE679M, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.

poulpreben commented 3 weeks ago

Det kommer lidt an på versionen om det præcist er linje 759. Kan du paste det, du har erstattet?

Du skal finde den linje hvor task["author"] indsættes i tekststrengen. Problemet (for mig) er at author ikke længere er med i task-objektet, så jeg har indført et check, som gør det til to linjer fremfor én.

ckref commented 3 weeks ago

Samme fejl her, som er løst ved @poulpreben metode

dennisroenn77 commented 3 weeks ago

Forsøgte @poulpreben løsningen, men uden held. Den virker ikke her.

Skærmbillede 2024-11-02 212617 Skærmbillede 2024-11-02 212448

poulpreben commented 3 weeks ago

Forsøgte @poulpreben løsningen, men uden held. Den virker ikke her.

Kan du bekræfte at du har genstartet efter du har foretaget ændringen? Hvis ja, kan du rulle ændringen tilbage, genstarte, og uploade nogle timers debug log? Dit problem må i så fald være et andet, så der er muligvis flere steder hvor vi med fordel kan bidrage med fejlhåndtering 😃

kaelken commented 3 weeks ago

Kan bekræfte løsningen som @poulpreben kom med, jeg får nu data ind. @dennisroenn77 Har du husket at rette begge linjer da 744 tager author i stedet for task["author"] som den gjorde før. Vil tro at det skulle virke for dig også, da vi har børn samme sted :D

Ronnie-J commented 3 weeks ago

Jeg har samme problem. Jeg har med @poulpreben rettelse fået den til at hente data igen fra Aula, men ikke ugeplaner. Der fejler den fortsat. Vi anvender Meebook til ugeplaner på skolen.

mjay9876 commented 3 weeks ago

Hej jeg har prøvet at gøre @poulpreben rettelse men får ingen data forsat

har vedhæfted debug og et par skærmudklip hvis der er en der kan hjælpe mig på rette vej :) config debug.txt Unavngivet

dennisroenn77 commented 3 weeks ago

Hej alle, Tak for god support :-) Efter jeg har indsat de 2 linier i client.py, har jeg genstartet hver gang. Endda afinstalleret både integration og HACS integrationen, men uden held. Nu prøvede jeg lige igen at "rense" det hele ud, genstarte og vente i 15 min. Herefter installerede jeg HACS integrationen igen, og så der lå en version 0.1.43 :-) Halleluja! Den tog jeg, MEN det virkede forsat ikke. :-( Jeg forsøgte med at indsætte de 2 linier @poulpreben foreslog, genstartede, og nu virker alt. Både kalender og ugeplan. Jeg går nårmalt ikke i kirke om søndagen, men måske burde jeg - for mine bønner er hørt.

Tak for en fantastik integration, nu kører det igen - i øvrigt med ChatGTP som oplæser :-)

Oceanfarm commented 3 weeks ago

Det kommer lidt an på versionen om det præcist er linje 759. Kan du paste det, du har erstattet?

Du skal finde den linje hvor task["author"] indsættes i tekststrengen. Problemet (for mig) er at author ikke længere er med i task-objektet, så jeg har indført et check, som gør det til to linjer fremfor én.

image

Oceanfarm commented 3 weeks ago

Hvis jeg fravælger "Tilføj ugeplaner som sensor attributter?", så virker alt undtagen ugeplaner.

Hvis jeg derimod tilvælger det, så virker intet.

Nedenstående er fejllog hvis jeg udelukkende tilvælger det.

`2024-11-04 09:31:25.286 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up aula platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform await asyncio.shield(awaitable) File "/config/custom_components/aula/sensor.py", line 75, in async_setup_entry await hass.async_add_executor_job(client.update_data) File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/aula/client.py", line 777, in update_data ugeplan(thisweek, "this") File "/config/custom_components/aula/client.py", line 760, in ugeplan r"([0-9]+)(.)", r"\1.", task["content"]


KeyError: 'content'`

Screenshot af aktuel kode:

<img width="690" alt="image" src="https://github.com/user-attachments/assets/af1f83a6-3f06-4510-870f-2a2702011c82">
mrryel commented 2 weeks ago

rettede client.py linje 744 (lidt forskelligt fra install til intall) til @poulpreben lille fix:

author = task.get('author', "") ugep = ugep + author + "

"

og vupti, så virkede det igen. Tak @poulpreben <3

SkinnyGnome commented 2 weeks ago

Hej alle, Tak for god support :-) Efter jeg har indsat de 2 linier i client.py, har jeg genstartet hver gang. Endda afinstalleret både integration og HACS integrationen, men uden held. Nu prøvede jeg lige igen at "rense" det hele ud, genstarte og vente i 15 min. Herefter installerede jeg HACS integrationen igen, og så der lå en version 0.1.43 :-) Halleluja! Den tog jeg, MEN det virkede forsat ikke. :-( Jeg forsøgte med at indsætte de 2 linier @poulpreben foreslog, genstartede, og nu virker alt. Både kalender og ugeplan. Jeg går nårmalt ikke i kirke om søndagen, men måske burde jeg - for mine bønner er hørt.

Tak for en fantastik integration, nu kører det igen - i øvrigt med ChatGTP som oplæser :-)

Jeg forsøgte med denne løsning, men uden held desværre.

Istedet for fejlen: File "/config/custom_components/aula/client.py", line 759, in ugeplan author = task.get('author', "")

Får jeg nu: Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform await asyncio.shield(awaitable) File "/config/custom_components/aula/sensor.py", line 75, in async_setup_entry await hass.async_add_executor_job(client.update_data) File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/aula/client.py", line 779, in update_data ugeplan(thisweek, "this") File "/config/custom_components/aula/client.py", line 762, in ugeplan r"([0-9]+)(.)", r"\1.", task["content"]**

Jeg har kigget efter om indrykninger osv. er korrekt og der ser fint ud.

Edit: Jeg forsøgte lige at kommentere denne del ud: image Så loader Aula Integrationen. Problemet er jo så bare at Content mangler i uge planen :-(

Jogge commented 2 weeks ago

Hvis I har fundet en løsning på problemet, vil det være en stor hjælp for projektet, hvis I opretter et Pull Request. Det er langt bedre end at sende andre på jagt efter, hvilke ændringer de skal lave selv – på den måde bliver løsningen tilgængelig for alle.

Nu bruger I jo selv dette open source-projekt, så det mindste, man kan gøre, er at give lidt tilbage i stedet for blot at holde løsningen for sig selv...

SkinnyGnome commented 2 weeks ago

Jeg fik leget lidt med debugging og fandt frem til fejlen med "task["content"]" Det viser sig at content ikke findes hvis task typen er "assignment".

Da assignement ikke indeholder key content mend title, har jeg indsat følgende.

                                        author = task.get('author', "")
                                        ugep = ugep + author + "<br><br>"
                                        if task["type"] == "comment" or task["type"] == "task":
                                            content = re.sub(
                                                r"([0-9]+)(\.)", r"\1\.", task["content"]
                                            )
                                        elif task["type"] == "assignment":
                                            content = re.sub(
                                                r"([0-9]+)(\.)", r"\1\.", task["title"]
                                            )
                                        ugep = ugep + content + "<br><br>"

image

Jeg når desværre nok ikke at lave Pull Requests hen over denne weekend, da den er maks presset i forvejen. @scaarup: Måske du kan se om overstående giver mening at smide ind?

poulpreben commented 2 weeks ago

Jeg afventede lige feedback før jeg ville oprette PR. Det er hermed gjort, inkl. @SkinnyGnome ændringer.

scaarup commented 1 week ago

Blev det her problem egentlig løst? PR er merged og der kom en ny version.

Jogge commented 1 week ago

Ja, problemet blev løst i #192.