search

deltaproject / Delta

📚 Een moderne versie van Magister, gemaakt voor leerlingen.
https://delta.keesvv.nl
Mozilla Public License 2.0
13 stars 4 forks source link

Inloggen mislukt #44

Closed QkeleQ10 closed 3 years ago

QkeleQ10 commented 5 years ago

Wanneer ik inlog, krijg ik de foutmelding dat mijn gegevens onjuist zijn.

keesvv commented 5 years ago

Hi @QkeleQ10! Leuk dat je mijn project hebt gevonden en de moeite hebt genomen om een ticket te openen! Dat is vervelend, als ik eraan toekom zal ik even ernaar kijken! Het zou namelijk kunnen zijn dat MagisterJS een nieuwe authcode heeft. De laatste tijd ben ik niet meer actief bezig met Delta, maar ik zal het onderzoeken! 😁

QkeleQ10 commented 5 years ago

Heel erg bedankt voor je razendsnelle reactie! Delta ziet er echt super geweldig uit. Het zou heel fijn zijn als ik eindelijk het slechte Magister kon opgeven en naar Delta kon.

From: Kees van Voorthuizen notifications@github.com Sent: Monday, September 30, 2019 1:40:51 PM To: deltaproject/Delta Delta@noreply.github.com Cc: QkeleQ10 quintenra@gmail.com; Mention mention@noreply.github.com Subject: Re: [deltaproject/Delta] Can't log in (kan niet inloggen) (#44)

Hi @QkeleQ10https://github.com/QkeleQ10! Leuk dat je mijn project hebt gevonden en de moeite hebt genomen om een ticket te openen! Dat is vervelend, als ik eraan toekom zal ik even ernaar kijken! Het zou namelijk kunnen zijn dat MagisterJS een nieuwe authcode heeft. De laatste tijd ben ik niet meer actief bezig met Delta, maar ik zal het onderzoeken! 😁

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/deltaproject/Delta/issues/44?email_source=notifications&email_token=ANLDWNHU5SALLCYMWXY5YW3QMHQUHA5CNFSM4I3ZVKSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD75LJMA#issuecomment-536523952, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANLDWNDSY7MYKKFGVXSNWJTQMHQUHANCNFSM4I3ZVKSA.

keesvv commented 5 years ago

Haha anytime! Dat is een groot compliment voor mij, dank je wel! 😄

QkeleQ10 commented 5 years ago

Hou me alsjeblieft up-to-date als het probleem is gefixt of als ik op een andere manier kan inloggen

keesvv commented 5 years ago

@QkeleQ10 Natuurlijk, zal ik doen! Deze twee dagen waren heel hectisch voor mij, maar ik zal straks even kijken waar het probleem ligt. Ik laat je weten wanneer het gefixt is! 😁

QkeleQ10 commented 5 years ago

Oké, heel erg bedankt!

QkeleQ10 commented 5 years ago

Ik bedoelde niet het Issue te sluiten haha

keesvv commented 5 years ago

Heheh geen probleem hoor! 😉

netlob commented 5 years ago

Ja MagisterJS heeft wat problemen. Sinds de nieuwe login van magister zijn er een aantal dingen veranderd. We zijn druk bezig het te verhelpen, maar het makkelijkste is om even terug te springen naar versie "^2.0.0-alpha.12" van MagisterJS (npm i magisterjs@2.0.0-alpha.12)

QkeleQ10 commented 5 years ago

Bedoel je dat ik dat zelf moet doen? Of word dat nu behind the scenes gedaan? Ben zelf niet zo goed met npm enzo...

From: Sjoerd Bolten notifications@github.com Sent: Thursday, October 3, 2019 5:40:40 PM To: deltaproject/Delta Delta@noreply.github.com Cc: QkeleQ10 quintenra@gmail.com; State change state_change@noreply.github.com Subject: Re: [deltaproject/Delta] Can't log in (kan niet inloggen) (#44)

Ja MagisterJS heeft wat problemen. Sinds de nieuwe login van magister zijn er een aantal dingen veranderd. We zijn druk bezig het te verhelpen, maar het makkelijkste is om even terug te springen naar versie "^2.0.0-alpha.12" van MagisterJS (npm i magisterjs@2.0.0-alpha.12)

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://github.com/deltaproject/Delta/issues/44?email_source=notifications&email_token=ANLDWNFU3P7QDOTHWAYBIF3QMYG7RA5CNFSM4I3ZVKSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAIT5VY#issuecomment-538001111, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANLDWNCFKMHCAUVQNISYU33QMYG7RANCNFSM4I3ZVKSA.

netlob commented 5 years ago

Nee. Dat zou de maker moeten doen en dan de app opnieuw distribueren :)

QkeleQ10 commented 5 years ago

Oh ok

keesvv commented 5 years ago

@QkeleQ10 @Netlob Als ik zometeen thuis ben zal ik een PR maken met de fix 😄

QkeleQ10 commented 5 years ago

Ik zag op de roadmap dat je plant om een nieuwe laad-animatie te maken. Ik heb een idee: ezgif-3-206811685cde Of misschien langzamer: ezgif-3-07abc2baaa30 Ik kan hem ook nog langzamer doen

Edit: misschien kun je, tijdens het laden, het stilstaande logo vervangen met het bewegende?

netlob commented 5 years ago

Ziet er sick uit!

keesvv commented 5 years ago

@QkeleQ10 Haha, huh?! Ik heb die niet gemaakt even voor de duidelijkheid, maar hij ziet er echt super gaaf uit! :smile: Die kunnen we misschien binnenkort implementeren.

EDIT: Hahah even opnieuw gelezen, ik had inderdaad zelf een roadmap item gemaakt regarding een nieuwe laadanimatie. Heb jij deze gemaakt?! Echt super slick gedaan!! :sweat_smile: Deze is geweldig, als we het authenticatieprobleem binnenkort hebben opgelost zal ik de langzame versie implementeren, echt heel cool!

keesvv commented 5 years ago

@QkeleQ10 @Netlob Even een update: Ik heb daarnet allerlei stappen genomen om het probleem te fixen, onder andere:

Helaas werkte geen van dit alles, en ik ben tot de conclusie gekomen dat het probleem dus bij MagisterJS/de Magister-API ligt. Hier kan ik zelf nu erg weinig aan veranderen, hopelijk kan ik binnenkort kijken of er een nieuwe authcode te halen valt. :grimacing:

QkeleQ10 commented 5 years ago

Heb jij deze gemaakt?!

Ja, dat heb ik zojuist gedaan! Als je ooit iets nodig hebt dat te maken heeft met graphic design, vraag het maar 😄

Helaas werkte geen van dit alles,

Dat is jammer, er is dus geen manier om in te loggen op het moment?

netlob commented 5 years ago

@keesvv oh vergeten te zeggen: magister heeft weer(...) een nieuw algoritme voor het genereren van de aiuthcode, de dynamic authcode parser werkt dus niet meer. Beste wat je kan doen is de authcode uit magister extracten. Denk aan een headless browser voor server apps, of (ik geloof dat dit react is) een request met een allow redirect erin.

Voorbeeld code: https://github.com/mamorgen/magbot3/blob/master/lib/magister/authcode.function.js success haha :)

keesvv commented 5 years ago

Super bedankt voor de input, @Netlob! Hahah hebben ze het alweer veranderd? Heb je code eventjes bekeken, ik zie dat je Puppeteer gebruikt, daar heb ik wel eens van gehoord. Ik snap het systeem: een browser window wordt gesimuleerd die naar de magisterpagina van je school gaat, en dan wacht deze op een request naar de username challenge (de browser heeft dan toegang om deze te bereiken). Ik zal dit weekend kijken of het mij lukt! :smile:

Even een andere update over dit project: ik kan Delta niet meer erg actief maintainen aangezien ik bezig ben aan een heel groot closed-source project op dit moment. Ik blijf uiteraard actief met issues en pull requests, maar zodra ik het inlogprobleem heb opgelost zal ik niet meer veel tijd besteden aan het vrijwillig schrijven van nieuwe code. Je bent altijd welkom om bijdragen te leveren, pull requests te maken etc. en ik zal er zo snel mogelijk naar kijken! Maar in de toekomst wil ik min of meer kijken of ik een nieuwe maintainer voor dit project kan vinden.

keesvv commented 5 years ago

Even een update: ik ben nog niet toegekomen aan een fix maken, omdat ik het gigantisch druk met school had dit weekend, ik zal proberen om het ergens deze week te fixen. Je bent altijd van harte welkom om een pull request te maken!

QkeleQ10 commented 5 years ago

Misschien kan @Netlob een pull request maken?

@keesvv oh vergeten te zeggen: magister heeft weer(...) een nieuw algoritme voor het genereren van de aiuthcode, de dynamic authcode parser werkt dus niet meer. Beste wat je kan doen is de authcode uit magister extracten. Denk aan een headless browser voor server apps, of (ik geloof dat dit react is) een request met een allow redirect erin.

Voorbeeld code: https://github.com/mamorgen/magbot3/blob/master/lib/magister/authcode.function.js success haha :)

QkeleQ10 commented 5 years ago

Nog enige updates? Ik ben heel enthousiast over Delta nu, het voelt als een grote opening waarop ik wacht 🎉😜😎

netlob commented 5 years ago

Heb zo een tussenuurtje en heb m'n laptop bij me. Zal kijken wat ik kan betekenen🙃

LevitatingBusinessMan commented 5 years ago

Haha, zo zie ik het graag. De beste commits zijn altijd tijdens schooluren geschreven.

netlob commented 5 years ago

image haha wat is dit nou. "hoogste cijfer een 100, gemiddelde een 194"

QkeleQ10 commented 5 years ago

Hahahaha lol

QkeleQ10 commented 5 years ago

image Wanneer je delta te vaak opnieuw start

keesvv commented 5 years ago

Hoi allemaal, ik had het deze twee weken echt onwijs druk met school en ik ben niet toegekomen aan een fix. Sorry als ik mensen zo lang heb laten wachten, ik heb inmiddels super veel dingen naast Delta te doen en het lukt mij niet meer om alle projecten en alles actief te houden. 🥴 @Netlob het ziet eruit alsof het is gelukt om in te loggen!! Echt geweldig, hoe heb je het uiteindelijk gefixt? 😁

keesvv commented 5 years ago

Ook even een quick note over de code: ik heb Delta geschreven toen ik nog maar net was begonnen met web development. Dus de code is echt een puinhoop geworden en als ik eerlijk ben snap ik er zelf ook helemaal niets meer van. 😝

Een tijd geleden is er een rewrite-branch gemaakt door @jvdoorn, waar hij Delta helemaal opnieuw heeft geschreven met een veel nettere codebase. Eventueel zouden we deze kunnen afmaken en vervolgens mergen, waardoor Delta iets meer maintainable wordt.

QkeleQ10 commented 5 years ago

Ik kan helaas nog steeds niet inloggen. Moet ik Delta verwijderen en weer installeren?

@Netlob op Studeo waren jouw cijfers ook raar aangegeven. Ligt dat misschien aan je school? Ik had daar geen problemen mee namelijk.

keesvv commented 5 years ago

@QkeleQ10 Het lijkt erop alsof @Netlob de code nog niet heeft gepusht, waardoor de nieuwe login alleen voor hem beschikbaar is. Als hij een pull request wil maken, zal ik het mergen en release ik een nieuwe versie van Delta. Dan kun je de oude verwijderen en de nieuwe downloaden van deltaproject.github.io! 😁

netlob commented 5 years ago

Oke. Probleem is gevonden: authcode heeft problemen. Wat eigenlijk de enige optie is om een headless browser te runnen die op een redirect naar https://accounts.magister.net/challenges/current afwacht en daar in de headers de authcode pakt. Alleen dat is dan weer een no-go omdat je niet een dependency van 100MB+ kan verplichten die een extra los programma opent voor 1 authurl. Oplossing zou zijn om een server te hosten met dit erop en dan daar zelf een call naar doen. Dus als je nog een server heb kan dat.

ps. heb 'm nu gefixt door zelf ff de authcode er in te hardcoden.

ps2. als je echt tijd teveel hebt kun je de oauth van magister implementeren, zie: https://www.youtube.com/watch?v=-mt21zyExd0

LevitatingBusinessMan commented 5 years ago

Het is me in t verleden gelukt om de authcode met een request te krijgen, en volgensmij heeft MagisterJS daar nu een aparte repo voor.

keesvv commented 5 years ago

@Netlob Top, super bedankt voor de info!! 😄 Heel nice dat het gelukt is om een updated authcode te vinden. Ik wist dat er wat problemen waren met de authcode, maar ik vond het zelf heel overkill om Puppeteer te installeren (zoals je al zegt: het is een super grote dependency).

Een hele server hosten die de authcodes dynamisch ophaalt lijkt mij al helemaal overkill, vooral voor een open source project, aangezien daar ook kosten aan verbonden zijn (ik heb zelf een server maar die is niet public). 😅

Misschien komt er ooit nog een library uit die dit nog makkelijker kan handelen, maar voor nu zouden we het misschien op deze manier moeten doen. In ieder geval, super tof dat je de tijd ervoor hebt genomen en als je wil kun je een PR maken zodat ik een nieuwe release kan maken met de nieuwe (waarschijnlijk tijdelijke) authcode.

keesvv commented 5 years ago

@LevitatingBusinessMan Klopt, die heb ik ook al geprobeerd. Daar zijn twee packages voor, en beiden werken ze niet meer met het nieuwe inlogsysteem 😬

keesvv commented 5 years ago

Oeps, dat ging verkeerd, sorry! 😅

netlob commented 5 years ago

Ja klopt die repos zijn verouderd. Ik heb nog een servertje draaien waar ik evt nog t op kan draaien.

Ohja en die codebase is een enorme pain in the ass.🤣

LevitatingBusinessMan commented 5 years ago

Misschien moet je dan even een issue maken, dan is het waarschijnlijk zo gefixt. MagisterJS is ook afhankelijk van die package.

LevitatingBusinessMan commented 5 years ago

Ik zou zelf er naar kijken, maar ik zit bij een open dag. Misschien moeten we ook even de aandacht van @idiidk erbij halen

idiidk commented 5 years ago

Yo, what's up? Dit is wel een lange thread moet ik zeggen :D

QkeleQ10 commented 5 years ago

Het enige wat ik weet is dat er iets mis is met de authcode van Delta

LevitatingBusinessMan commented 5 years ago

Van wat ik ervan begrepen heb, authcode kapoet.

idiidk commented 5 years ago

Oke dit kan best kloppen, ik weet dat er een fout in m'n magister-auth package zit waardoor die de authcode niet van Magister pakt. Was al bezig met een update, denk dat ik vanmiddag wel ff een pull request kan maken

netlob commented 5 years ago

was vergeten de PR aan te maken, dus heb dat net even gedaan. (https://github.com/deltaproject/Delta/pull/45)

delta werkt weer 🎉

notes: Little hotfix using my own server. The authcode at http://78.47.72.123/code.json is updated every minute, which should be fine right now.

keesvv commented 5 years ago

Hi @Netlob, echt super bedankt voor je PR!! Geweldig dat je de tijd hebt willen nemen om Delta weer aan de praat te krijgen, echt top! :smile: :tada: Ik zal hem gelijk mergen, en dan release ik de nieuwe versie! :+1: :smile:

keesvv commented 5 years ago

Ik heb een nieuwe release aangemaakt voor Delta! Deze kun je hier vinden. Super bedankt allemaal voor de support en bijdragen, appreciate it 😁

EDIT: Wacht nog even met het downloaden van de release, zie dat ik een foutje in package.json heb gemaakt

EDIT 2: Je kunt hem downloaden hoor! Was per ongeluk vergeten het versienummer te updaten waardoor Delta steeds ging zoeken naar een nieuwere versie.

netlob commented 5 years ago

image

keesvv commented 5 years ago

Hmm, dat is apart. Lijkt er bijna op alsof de node_modules niet mee zijn gekomen in de release. Zal er straks wel even naar kijken!

keesvv commented 5 years ago

Probleem is opgelost! Bleek dat alleen dependencies waren meegenomen, zonder devDependencies. Dit hoort natuurlijk niet zo te werken, maar nu heb ik Delta gepackaged wél met de devDependencies. Hier is de nieuwe release :smile: