scaarup / aula

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

No entries are provided by the plugin anymore #114

Closed ricobach closed 11 months ago

ricobach commented 11 months ago

Describe the bug or problem (In Danish is OK) No entries are provided by the plugin anymore, in the log i can see all children and the school and institutions..
A clear and concise description of what the bug is. --> im not sure if its related to the change og aula API from v15 to v16

Please answer the following

Please provide debug log from the integration

scaarup commented 11 months ago

Prøv lige at hop ind på Aula på din telefon/web og se om du har ulæste beskeder. Marker dem som læst og genstart din HA og se om samme fejl opstår.

ricobach commented 11 months ago

Jeg har som regel flere ulæste beskeder da der er meget "spam", men der var komme en som var markeret som "følsom information" da jeg læste den virkede det igen..

Tusinde tak :D

scaarup commented 11 months ago

Ah interessant :) Det lyder som du har ramt en bug. Hvis du gider, må du gerne markere den som ulæst. Indkommentere linje 198 i client.py og genstarte HA. Der er nok en eller anden edge-case med "følsom information", jeg ikke håndterer korrekt i integrationen.

ricobach commented 11 months ago

Ligeså snart at jeg markere den som ulæst. kom fejlen tilbage... jeg nåede ikke at fange loggen og når jeg prover lige nu er Aula ikke tilgængelig.. jeg prøver når aula er oppe igen..

ricobach commented 11 months ago

Når den første/nyeste besked som er ulæst er "følsom information" kommer fejlen(se log). hvis jeg markere en besked før som ulæst eller markere den følsomme besked som læst, virker alt som normalt..

aulalog3.txt

scaarup commented 11 months ago

Men fik du indkommenteret den linje jeg nævnte?

ricobach commented 11 months ago

Ja, den er indkommenteret, og når den ikke tager fat i den følsomme besked kommer der også en masse ekstra log.. jeg har lige lagt mærke til at hvis jeg logger ind på aula fra en browser med unilogin kræver den MitID login..

Screenshot 2023-09-03 at 20 14 26 Screenshot 2023-09-03 at 20 17 41
scaarup commented 11 months ago

Fedt, det var bare mig, der overså det.

Det her leder vi efter: 2023-09-03 12:03:44.889 DEBUG (SyncWorker_11) [custom_components.aula.client] threadres {"status":{"code":403,"subCode":8,"errorInformation":[],"message":""},"data":null,"trace":"n\/a"}

Så jeg mangler at håndtere den i integrationen. Tak :)

scaarup commented 11 months ago

@ricobach det kan være du vil teste den beta/pre-release, jeg lige har lagt ud? Selvfølgelig gerne med følsom besked som ulæst.

ricobach commented 11 months ago

Jeg har testet med følsom besked som ulæst og det virker som jeg ville forvente..

scaarup commented 11 months ago

Okay tak. Så betragter vi det som en løsning.