User auth #3 ready #71

Avasam · 2017-04-15T23:34:52Z

No description provided.

First step towards a working login. The login finds the approriate user in the database. Next step would be to compare the hashed password. Then to establish a continuous connection with the user. [#3] authentification: User can login An invalid login will reload /login page. A valid login will redirect the user to the homepage. AFAIK the user will stays connected for a period of 3 days. Proper login/logout detection. Separate Login from Signup in LoginHandler Very first working register/login/logout Current user datas now accessible Client side verification. First try at client side verification. Added field for password verification Client side validation Password confirmation General use of $http Client side Hash+Salt +tool Let user Signup and Login without email +login/signup ++login/signup +++ 'name' is a reserved word in Javascript

<WIP> 3rd party auth (not yet keeping credentials) Basic Facebook Auth Basic Google+ Auth Path changes <WIP> working Twitter Auth Working Twitter Auth Working Twitter Auth + Twitter better try-except Working Facebook Auth + Facebook comment fix Working Google+ Auth + google fix

Bower - Font Awesome fix Cleaned js <script> by removing duplicate It was previously needed to write a js <script> tag twice if used both with and without the debug option. +

mathben · 2017-04-29T17:21:54Z

src/web/handlers.py

@@ -107,9 +284,9 @@ def get(self):
        if self._global_arg["disable_character"]:
            return

-        player_id = self.request.query[len("player_id="):]
+        user_id = self.request.query[len("user_id="):]


Il va falloir migrer la base de donnée avec les nouvelles clés utilisés ou reprendre les anciennes clés et les modifier plus tard pour des noms plus juste.

player_id -> user_id

DONE - ce n'est plus pertinant dans le contexte qu'on efface la base de donnée du passé.

mathben · 2017-04-29T17:23:09Z

src/web/handlers.py

+        email = self.get_argument("email", default=None)
+
+        if name:
+            self.write("0" if (self._db.user_exists(name=name) or self._db.user_exists(email=name)) else "1")


pourquoi ne pas retourner un json?

Puis mettre une boolean au lieu de "0" ou "1"?

DONE - ça va être bon pour le moment. La requête est envoyé à chaque caractère. Ça doit être optimisé dans le future.

mathben · 2017-04-29T17:24:09Z

src/web/__main__.py

@@ -14,6 +14,7 @@
 DB_DEMO_PATH = os.path.join("..", "..", "database", "demo_user.json")
 DB_MANUAL_PATH = os.path.join("..", "..", "database", "tl_manual.json")
 DB_LORE_PATH = os.path.join("..", "..", "database", "tl_lore.json")
+DB_KEYS_PATH = os.path.join("..", "..", "database", "keys.json")


key.json n'est pas un nom significatif

"Contains keys and secrets needed for third-party authentification."
Pourquoi pas le renommer auth.json?

DONE, renommé pour auth.json

mathben · 2017-04-29T17:27:11Z

src/web/handlers.py

+            user = self._db.get_user(user_id=user_id)
+        else:
+            user = self.current_user
+        self.render('profile.html', user=user, **self._global_arg)


serait-ce mieux renommer profile.html pour user_profile.html ?
Car dans le contexte, il peut avoir d'autre profile comme celui du character.

DONE On abandonne pour le moment ce commentaire.

mathben · 2017-04-29T17:28:27Z

src/web/handlers.py

+
+        # Produce a missing argument error
+        else:
+            self.get_argument("username or email")


Est-ce que c'est ceci qu'il faut aller chercher? «username_or_email»

De toute façon, on ne fait rien avec. On veut créer une erreur? Je ne comprend pas la raison de cette ligne.

DONE - Via l'interface usagé web, on ne peut pas se rendre à cette étape. Seul des tests unitaires ou des requêtes http en ne passant pas via l'interface usagé pourra exécuter cette ligne. Un todo a été ajouté.

mathben · 2017-04-29T17:37:20Z

src/web/py_class/db.py

-            return _user
+    def user_exists(self, email=None, user_id=None, name=None):
+        """Returns True if all the arguments given are found"""
+        return(not(email and not self._db_user.get(self._query_user.email == email)) and


première () est de trop. Inutile même pour la compréhension.

mathben · 2017-04-29T17:38:20Z

src/web/py_class/db.py

-            print("Cannot update player if player is not dictionary : %s" % player_data)
+    def update_user(self, user_data, character_data=None, delete_user_by_id=None, delete_character_by_id=None):
+        if not isinstance(user_data, dict):
+            print("Cannot update user if user is not dictionary : %s" % user_data)
            return
        d = datetime.datetime.utcnow().timestamp()


Il faudrait renommer cette variable, d.

mathben · 2017-04-29T17:40:00Z

src/web/py_class/db.py

-            self._db_user.insert(player_data)
-        elif player_data or character_data or delete_character_id:
+            # TODO validate user_data field
+            user_data["user_id"] = uuid.uuid4().hex


On génère déjà plus tôt cette valeur dans create user. On devrait au lieu appeler la fonction create user?

Cancel, ce n'est pas une bonne idée.

mathben · 2017-04-29T17:41:22Z

src/web/resources/css/_base_dark.css

+  templates compile.  The !important is important given that there may be 
+  other selectors that are more specific or come later and might alter display.  
+ */
+[ng\:cloak], [ng-cloak], .ng-cloak {


dupliqué dans _base.css
Toutes les pages devraient avoir accès à _base.css

DONE
C'est enlevé. En fait, il existe déjà dans angularjs.min

mathben · 2017-04-29T17:43:13Z

src/web/web.py

+            # Generate a self-signed certificate and key if we don't already have one.
+            cmd = 'openssl req -x509 -sha256 -newkey rsa:2048 -keyout %s -out %s -days 36500 -nodes -subj' % (
+                KEY_FILE_SSL, CERT_FILE_SSL)
+            cmd_in = cmd.split() + ["/C=CA/ST=QC/L=Montreal/O=Traitre-lame/OU=gn.qc.ca"]


modifier gn.qc.ca pour l'adresse du grandeur nature actuel.

Ignorer, va être fait dans une autre branche avec l'intégration du ssl.

mathben · 2017-06-05T03:21:16Z

src/web/py_class/keys.py

+import json
+from sys import stderr
+
+class Keys(object):


Il faudrait un nom plus significatif. Une classe Keys c'est flou comme nom.

DONE - renommé pour auth_keys.py

mathben · 2017-06-05T03:23:45Z

src/web/py_class/keys.py

+            print("ERROR: "+parser.db_keys_path+" isn't formatted properly. \n"
+                 +"Details: "+str(exception), file=stderr)
+        except FileNotFoundError as exception:
+            print("WARNING: "+str(exception)+" \nA file has been created for you.")


Quel est l'utilité de créer ce fichier? Informer l'utilisateur qu'il doit remplir ce fichier. Habituellement, on fait un fichier template qu'on dit de copier via un README en décrivant comment faire.

DONE - enlevé du code, le readme va être écrit bientôt.

mathben · 2017-06-05T03:25:02Z

src/web/resources/js/tl_module/login_ctrl/login_ctrl.js

+          }).then(
+            function(response) {
+              console.log(response["data"]);
+              if (response["data"] == "0") {


Ceci est en lien avec le return "0" du serveur. À noter qu'il faudrait changer le type de retour.

Il a une faille de sécurité, on valide juste le "0", mais pas le "1". Le serveur pourrait retourner n'importe quoi et il va croire qu'il a réussi l'authentification.

mathben · 2017-06-05T03:41:29Z

src/web/web.py

+
+        ssl_options = {"certfile": CERT_FILE_SSL, "keyfile": KEY_FILE_SSL}
+
+    url = "http{2}://{0}:{1}".format(parse_arg.listen.address, parse_arg.listen.port, "s" if ssl_options else "")


L'URL est par défaut sur 127.0.0.1, ce n'est pas bon dans le cas du serveur en production.
À voir commencer changer ça de manière propre.
C'est utiliser pour les AUTH pour faire des redirections vers la bonne page. Sommes-nous obligés de mettre l'adresse?

DONE - Mis dans un commentaire dans la tâche #3 , il faudrait pour le moment utiliser un fichier de config et spécifier l'adresse qui sera par défaut le 127.0.0.1 + port

mathben · 2017-06-05T03:46:32Z

src/web/partials/profile.html

+{% if user %}
+<div>This is {{user.get("name")}}'s profile page. (ID: {{user.get("user_id")}})</div>
+</br>
+<div>Here the user should be able to modify certain personnal infos as well as his name, e-mail and password.</div>


Devrait être mis en français, le site est en français pour le moment.

mathben · 2018-02-23T08:49:11Z

src/web/partials/login.html

          <br/>

-          <button class="btn btn-lg btn-primary btn-block" type="submit">Login</button>
+          <button class="btn btn-lg btn-primary btn-block" type="submit" onclick="if(username_or_email.value && loginForm.password.value){password.value=hashSha256(password.value, username_or_email.value);};">Se connecter</button>


This can be more secure, the hash is password with email. Easy to find password ...

Cancel, it need to be binded with angularjs. Added comment.

mathben · 2018-02-23T08:49:31Z

src/web/partials/login.html

+          <span class="help-block" ng-show="signUpForm.pwconfirm.$error.fieldMatch && signUpForm.pwconfirm.$dirty">Le mot de passe n'est pas identique.</span>
+          <br/>
+
+          <button class="btn btn-lg btn-primary btn-block" type="submit" ng-disabled="signUpForm.$invalid || signUpForm.$pending" onclick="var tempPW=password.value; password.value=hashSha256(tempPW, username.value);pwconfirm.value=hashSha256(tempPW, email.value);">Créer un compte</button>


This is OK, but can be improve.

mathben · 2018-03-04T04:23:09Z

src/web/__main__.py

@@ -14,6 +14,7 @@
 DB_DEMO_PATH = os.path.join("..", "..", "database", "demo_user.json")
 DB_MANUAL_PATH = os.path.join("..", "..", "database", "tl_manual.json")
 DB_LORE_PATH = os.path.join("..", "..", "database", "tl_lore.json")
+DB_KEYS_PATH = os.path.join("..", "..", "database", "keys.json")


DONE, renommé pour auth.json

mathben · 2018-03-04T04:30:10Z

src/web/handlers.py

+            user = self._db.get_user(user_id=user_id)
+        else:
+            user = self.current_user
+        self.render('profile.html', user=user, **self._global_arg)


DONE On abandonne pour le moment ce commentaire.

mathben · 2018-03-04T04:30:34Z

src/web/handlers.py

@@ -107,9 +284,9 @@ def get(self):
        if self._global_arg["disable_character"]:
            return

-        player_id = self.request.query[len("player_id="):]
+        user_id = self.request.query[len("user_id="):]


DONE - ce n'est plus pertinant dans le contexte qu'on efface la base de donnée du passé.

mathben · 2018-03-04T05:41:21Z

src/web/py_class/db.py

-            self._db_user.insert(player_data)
-        elif player_data or character_data or delete_character_id:
+            # TODO validate user_data field
+            user_data["user_id"] = uuid.uuid4().hex


Cancel, ce n'est pas une bonne idée.

mathben · 2018-03-04T05:41:55Z

src/web/py_class/keys.py

+import json
+from sys import stderr
+
+class Keys(object):


DONE - renommé pour auth_keys.py

mathben · 2018-03-04T05:43:06Z

src/web/py_class/keys.py

+            print("ERROR: "+parser.db_keys_path+" isn't formatted properly. \n"
+                 +"Details: "+str(exception), file=stderr)
+        except FileNotFoundError as exception:
+            print("WARNING: "+str(exception)+" \nA file has been created for you.")


DONE - enlevé du code, le readme va être écrit bientôt.

mathben · 2018-03-04T05:43:42Z

src/web/web.py

+            # Generate a self-signed certificate and key if we don't already have one.
+            cmd = 'openssl req -x509 -sha256 -newkey rsa:2048 -keyout %s -out %s -days 36500 -nodes -subj' % (
+                KEY_FILE_SSL, CERT_FILE_SSL)
+            cmd_in = cmd.split() + ["/C=CA/ST=QC/L=Montreal/O=Traitre-lame/OU=gn.qc.ca"]


Ignorer, va être fait dans une autre branche avec l'intégration du ssl.

mathben · 2018-03-04T05:47:09Z

src/web/web.py

+
+        ssl_options = {"certfile": CERT_FILE_SSL, "keyfile": KEY_FILE_SSL}
+
+    url = "http{2}://{0}:{1}".format(parse_arg.listen.address, parse_arg.listen.port, "s" if ssl_options else "")


DONE - Mis dans un commentaire dans la tâche #3 , il faudrait pour le moment utiliser un fichier de config et spécifier l'adresse qui sera par défaut le 127.0.0.1 + port

mathben · 2018-03-04T05:48:19Z

Close Pull Request. Review done. Reported in a new Pull Request.

Avasam added 4 commits April 15, 2017 18:59

Added a user profile page

1525cd9

Small HTML and Javascript fixes.

3f4e96b

Bower - Font Awesome fix Cleaned js <script> by removing duplicate It was previously needed to write a js <script> tag twice if used both with and without the debug option. +

mathben mentioned this pull request Apr 29, 2017

justletsencrypt.sh #73

Closed

mathben reviewed Apr 29, 2017

View reviewed changes

mathben reviewed Jun 5, 2017

View reviewed changes

mathben reviewed Mar 4, 2018

View reviewed changes

mathben closed this Mar 4, 2018


		ssl_options = {"certfile": CERT_FILE_SSL, "keyfile": KEY_FILE_SSL}

		url = "http{2}://{0}:{1}".format(parse_arg.listen.address, parse_arg.listen.port, "s" if ssl_options else "")

User auth #3 ready #71

User auth #3 ready #71

Conversation

Avasam commented Apr 15, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mathben commented Mar 4, 2018