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

Uitloggen verbeterd #14

Closed jvdoorn closed 5 years ago

jvdoorn commented 5 years ago

In plaats van de app opnieuw op te starten heb ik ervoor gezorgd dat de authenticatie status wordt gereset.

keesvv commented 5 years ago

Top! Ik zal er straks ff naar kijken als ik thuis ben, ik ben momenteel niet thuis.

keesvv commented 5 years ago

By the way, ik zag dat je een commit had gemaakt met de message dat de <i> tags overbodig waren, deze zijn in feite niet overbodig, deze zorgen voor een animatie wanneer er wordt gezocht naar een update.

jvdoorn commented 5 years ago

Mocht het nodig zijn dan zetten we ze terug. Zou je kunnen wachten met mergen? Ik zie zojuist dat Delta wachtwoorden opslaat in plain text, ik ga kijken of we het zo kunnen doen dat ze in de native password managers worden opgeslagen.

jvdoorn commented 5 years ago

Ik kon zo snel geen manier vinden om de gebruikersnaam en het wachtwoord veilig op te slaan. Echter staat er in magister.js een functie beschreven waarmee ingelogt kan worden d.m.v. een token i.p.v. een gebruikersnaam/wachtwoord. Wellicht is het mogelijk om dat te gebruiken? https://github.com/simplyGits/MagisterJS/blob/master/src/magister.js#L509

keesvv commented 5 years ago

Daar zal ik wel even naar kijken binnenkort. Deze week heb ik het heel druk met school, dus ik zal kijken wat ik kan doen 😁

keesvv commented 5 years ago

Oh wow ik heb net de PR bekeken, en ik vind het echt perfect dat je Delta nu eindelijk niet meer hoeft te restarten als je wilt afmelden. Dat is een pluspunt voor de gebruikerservaring, dus heel erg bedankt! Ik ga hem zo dadelijk mergen, morgen zal ik een release voor versie 0.6.6 uitbrengen :smile:

keesvv commented 5 years ago

Zodra ik een goede manier heb gevonden om credentials op te slaan zal ik een nieuwe versie aanmaken, tot nu toe was de authenticatie erg onveilig 😬