tomkooij/aanmeldr

Eenvoudige website voor het kiezen van workshops voor leerlingen

aanmeldr's Issues

Test User database still uses md5

Gebruik HTTPS

Logins worden nu in cleartext gepost. Gebruik HTTPS. Wordt in principe ondersteund door pythonanywhere en Flask met pyOpenSSL

alle database calls clusteren in een query_db() en write_db()

De code is een zooitje!

alle database calls clusteren in een query_db() en write_db() --> Contextmanager!

Vooral in db_tools() is dit heel erg nodig.

sla unix timestamp op bij inschrijving

Als een workshop (te) vol zit dan is het onduidelijk (moeilijk te achterhalen) wie als eerste en wie als laatste heeft ingeschreven.

Oplossing: Sla unixtimestamp op bij inschrijving (evt in de user tabel als "laatste inschrijving")

Een "bedankt voor het inschrijven. Je kunt nu uitloggen" bericht ontbreekt

Veel gebruikers kiezen een workshop en proberen daarna dezelfde workshop nog een of twee keer te kiezen. Blijkbaar is het niet duidelijk dat ze al ingeschreven en/of klaar zijn.

Toevoegen: Een "bedankt voor het inschrijven. Je kunt nu uitloggen" bericht.

Kies knop moet verander keuze worden zodra keuze gemaakt is

Dat dus

Fouten bij lezen/schrijven database afvangen

Voor bij schrijven database moeten we controleren op fouten en actie ondernemen.
Dwz: in kies_workshop()

Fix database errors

SQL transaction level is REPEATABLE READ

Dit betekent dat een SELECT altijd hetzelfde oplevert, ondanks dat er ondertussen iets is gewijzigd. http://dev.mysql.com/doc/refman/5.7/en/innodb-transaction-isolation-levels.html#isolevel_repeatable-read

Als er tijdens het inschrijven ondertussen minder plaatsen in een workshop zijn, dan zie je dat niet in een SELECT, die geeft nog de oude waarde.

Oplossing zou kunnen zijn: Lees aantal plaatsen na db.commit() tijdens het inschrijven en kijk of het niet meer klopt. Als het niet meer klopt: Schrijf uit en geef error.

mysql> SELECT * FROM information_schema.session_variables
    -> WHERE variable_name = 'tx_isolation';
+---------------+-----------------+
| VARIABLE_NAME | VARIABLE_VALUE  |
+---------------+-----------------+
| TX_ISOLATION  | REPEATABLE-READ |
+---------------+-----------------+
1 row in set (0.00 sec)
mysql> SELECT * FROM information_schema.global_variables
    -> WHERE variable_name = 'tx_isolation';
+---------------+-----------------+
| VARIABLE_NAME | VARIABLE_VALUE  |
+---------------+-----------------+
| TX_ISOLATION  | REPEATABLE-READ |
+---------------+-----------------+
1 row in set (0.00 sec)

Aanmeldr offline

ik krijg "De aanmeldwebsite is op dit moment niet open." op mijn scherm als ik probeer in te loggen.

Vang illegale workshop inschrijving via eigen gemaakte post af

Iemand kan zich inschrijven voor een woordkeuze door vol zit door zelf een post request te sturen. Het aantal ingeschreven deelnemers wordt dan negatief.

Twee dingen

• check op vol in mis_workshop()
• check op negatief aantal plaatsen bij inschrijven.