scaarup / aula

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

Login for medarbejder+forældre #24

Closed sorentorp closed 1 year ago

sorentorp commented 1 year ago

Super godt arbejde!

Når en person både er medarbejder og forældre med uni-login, virker denne integration ikke. Jeg formoder at det handler om denne mini-broker, som afkræver et rollevalg. Måske det kan indarbejdes? Jeg vedlægger den url jeg lander på, screenshot og kildekoden til siden.

https://broker.unilogin.dk/auth/realms/broker/login-actions/post-broker-login?client_id=https%3A%2F%2Fprod-uni.aula.dk%2Fsimplesaml%2Fsaml2%2Fidp%2Fmetadata.php&tab_id=QfHrn1LoZZw

Pastebin med kildekoden

Screenshot 2022-11-12 16 36 18

Skriv endelig hvis der er mere data jeg kan hjælpe med.

scaarup commented 1 year ago

Hejsa.

Vil du prøve med denne her branch: https://github.com/scaarup/aula/tree/24-login-for-medarbejder%2Bfor%C3%A6ldre Og slå debug logging til på denne integration.

Det er lidt et long-shot, når man ikke lige kan teste det selv ;).

sorentorp commented 1 year ago

Done - håber det kan bruges.

Debug 1

Debug 2

Debug 3

scaarup commented 1 year ago

Der er desværre ikke en eneste debug log linje. Vil du slå debug på integrationen, genstart HA og give mig alt det, der står aula på?

logger:
  default: info
  logs:
    custom_components.aula: debug
sorentorp commented 1 year ago

synes ellers jeg havde aktiveret debug, men din metode virker :-)

debug

scaarup commented 1 year ago

Gider du pull'e de seneste ændringer på branchen (https://github.com/scaarup/aula/tree/24-login-for-medarbejder%2Bfor%C3%A6ldre) og prøve igen?

sorentorp commented 1 year ago

Så er der nyt - no luck...

debug

scaarup commented 1 year ago

Tror ikke du har fået hentet den branch. Der står "Aula integration, version: 0.1.15" i din log, og jeg har rettet versionen til "0.1.15b".

sorentorp commented 1 year ago

rigtigt - nu fik jeg den ned. Det er den ikke glad for.

2022-11-13 00:08:32.160 ERROR (SyncWorker_0) [homeassistant.loader] The custom integration 'aula' does not have a valid version key (0.1.15b) in the manifest file and was blocked from loading. See https://developers.home-assistant.io/blog/2021/01/29/custom-integration-changes#versions for more details

scaarup commented 1 year ago

Ah okay. Så kan du selv lige lave versions strengen om.

sorentorp commented 1 year ago

Tjek - den blev bare til 0.1.15 :-)

debug

scaarup commented 1 year ago

Jeg kan ikke finde den form i noget af det vi får serveret.

Jeg har opdateret branch'en med meget mere debug log, så må du gerne lige prøve igen. Vi logger al HTML ud, så det bliver nok til en del.

sorentorp commented 1 year ago

Det blev da til en smule :-)

debug

scaarup commented 1 year ago

Tak :)

Det er meget meget svært når jeg ikke kan teste det her selv.

Er du hjemme i Developer tools i din browser? Kan du mon fange (copy as curl), den POST hvor du vælger, at du er logger ind som forældre?

sorentorp commented 1 year ago

ja da - tror det er 2'eren du er ude efter.

nr 1 nr 2 nr 3 nr 4 nr 5

scaarup commented 1 year ago

Perfekt tak.

Vil du prøve at opdatere til de seneste ændringer fra branchen (https://github.com/scaarup/aula/tree/24-login-for-medarbejder%2Bfor%C3%A6ldre)? Version skulle gerne sige "0.1.1337" når du starter den op.

sorentorp commented 1 year ago

så er der nyt

debug

scaarup commented 1 year ago

Jeg har ikke flere skud i bøssen...

Med mindre jeg kan få fat på et login som dit, kan jeg ikke rigtig komme videre desværre.

sorentorp commented 1 year ago

Hvis du har tid i morgen over middag, kan vi evt sætte et lille møde op? Skriv på "soren . Torp @ Gmail . Com"

scaarup commented 1 year ago

Jeg tror desværre der skal mere end et møde til. Det kræver, at man bruger en hel del tid på at reverse hele login flowet. En masse tests og en masse tid.

sorentorp commented 1 year ago

Okay, det er jo fair nok. De fleste der arbejder i en skole eller en børnehave har vel en bedre halvdel, som ikke gør 😀

JBoye commented 1 year ago

Har i set: https://github.com/JBoye/HA-Aula/issues/4#issuecomment-1007291754

Det lyder til at de knækkede den

scaarup commented 1 year ago

Har i set: https://github.com/JBoye/HA-Aula/issues/4#issuecomment-1007291754

Det lyder til at de knækkede den

Ah ja, det var da interessant 🙂. @sorentorp vil du give det et skud?

sorentorp commented 1 year ago

@scaarup er der et nyt branch jeg skal teste, eller hvad er det jeg skal give et skud? :-)

scaarup commented 1 year ago

@scaarup er der et nyt branch jeg skal teste, eller hvad er det jeg skal give et skud? :-)

Det var specifikt denne løsning, der var refereret til: image

Ændre user_data i client.py til:

user_data = { 'username': self._username, 'password': self._password, 'selected-aktoer': "KONTAKT" }
sorentorp commented 1 year ago

Det virker :-)

scaarup commented 1 year ago

Det virker :-)

Perfekt 👍

scaarup commented 1 year ago

Der er et fix med i v0.1.21, som er ude i pre-release nu.