Merge branch 'new-passwordless-spike' into 'master'

completed 18.1-18.3 See merge request !10

Merge branch 'new-passwordless-spike' into 'master'
completed 18.1-18.3 See merge request !10
37c2743b · Rahmania Astrid Mochtar · 7869d8ba · 34a41fa2 · 37c2743b · 37c2743b
Commit 37c2743b authored Nov 14, 2019 by Rahmania Astrid Mochtar
--- a/.gitignore
+++ b/.gitignore
@@ -3,3 +3,7 @@ __pycache__
 *.pyc
 /static
 env/
+
+.env
+geckodriver.log
+.vscode
\ No newline at end of file
--- a/accounts/__init__.py
+++ b/accounts/__init__.py
--- a/accounts/admin.py
+++ b/accounts/admin.py
+from django.contrib import admin
+
+# Register your models here.
--- a/accounts/apps.py
+++ b/accounts/apps.py
+from django.apps import AppConfig
+
+
+class AccountsConfig(AppConfig):
+    name = 'accounts'
--- a/accounts/authentication.py
+++ b/accounts/authentication.py
+import sys
+from accounts.models import ListUser, Token
+
+class PasswordlessAuthenticationBackend(object):
+
+    def authenticate(self, uid):
+        print('uid', uid, file=sys.stderr)
+        if not Token.objects.filter(uid=uid).exists():
+            print('no token found', file=sys.stderr)
+            return None
+        token = Token.objects.get(uid=uid)
+        print('got token', file=sys.stderr)
+        try:
+            user = ListUser.objects.get(email=token.email)
+            print('got user', file=sys.stderr)
+            return user
+        except ListUser.DoesNotExist:
+            print('new user', file=sys.stderr)
+            return ListUser.objects.create(email=token.email)
+
+
+    def get_user(self, email):
+        return ListUser.objects.get(email=email)
\ No newline at end of file
--- a/accounts/migrations/0001_initial.py
+++ b/accounts/migrations/0001_initial.py
-# Generated by Django 2.1.7 on 2019-11-13 14:32
+# Generated by Django 2.1.7 on 2019-11-13 16:08

 from django.db import migrations, models


--- a/accounts/migrations/__init__.py
+++ b/accounts/migrations/__init__.py
--- a/accounts/models.py
+++ b/accounts/models.py
+from django.db import models
+from django.contrib.auth.models import (
+    AbstractBaseUser, BaseUserManager, PermissionsMixin
+)
+
+
+class Token(models.Model):
+    email = models.EmailField()
+    uid = models.CharField(max_length=255)
+
+class ListUserManager(BaseUserManager):
+
+    def create_user(self, email):
+        ListUser.objects.create(email=email)
+
+    def create_superuser(self, email, password):
+        self.create_user(email)
+
+class ListUser(AbstractBaseUser, PermissionsMixin):
+    email = models.EmailField(primary_key=True)
+    USERNAME_FIELD = 'email'
+    #REQUIRED_FIELDS = ['email', 'height']
+
+    objects = ListUserManager()
+
+    @property
+    def is_staff(self):
+        return self.email == 'harry.percival@example.com'
+
+    @property
+    def is_active(self):
+        return True
+
--- a/accounts/templates/login_email_sent.html
+++ b/accounts/templates/login_email_sent.html
+<html>
+<h1>Email sent</h1>
+
+<p>Check your email, you'll find a message with a link that will log you into
+the site.</p>
+
+</html>
\ No newline at end of file
--- a/accounts/tests.py
+++ b/accounts/tests.py
+from django.test import TestCase
+
+# Create your tests here.
--- a/accounts/tests/test_models.py
+++ b/accounts/tests/test_models.py
-from django.test import TestCase
-from django.contrib.auth import get_user_model
-
-from accounts.models import Token
-
-User = get_user_model()
-
-
-class UserModelTest(TestCase):
-
-    def test_user_is_valid_with_email_only(self):
-        user = User(email='a@b.com')
-        user.full_clean()  # should not raise
-
-    def test_email_is_primary_key(self):
-        user = User(email='a@b.com')
-        self.assertEqual(user.pk, 'a@b.com')
-
-
-class TokenModelTest(TestCase):
-
-    def test_links_user_with_auto_generated_uid(self):
-        token1 = Token.objects.create(email='a@b.com')
-        token2 = Token.objects.create(email='a@b.com')
-        self.assertNotEqual(token1.uid, token2.uid)
--- a/accounts/urls.py
+++ b/accounts/urls.py
+from django.conf.urls import url
+from accounts import views
+
+urlpatterns = [
+    url(r'^send_email$', views.send_login_email, name='send_login_email'),
+    url(r'^login$', views.login, name='login'),
+    url(r'^logout$', views.logout, name='logout'),
+]
\ No newline at end of file
--- a/accounts/views.py
+++ b/accounts/views.py
+import uuid
+import sys
+from django.shortcuts import render, redirect
+from django.core.mail import send_mail
+from django.contrib.auth import authenticate
+from django.contrib.auth import login as auth_login, logout as auth_logout
+
+from accounts.models import Token
+
+
+def send_login_email(request):
+    email = request.POST['email']
+    uid = str(uuid.uuid4())
+    Token.objects.create(email=email, uid=uid)
+    print('saving uid', uid, 'for email', email, file=sys.stderr)
+    url = request.build_absolute_uri(f'/accounts/login?uid={uid}')
+    send_mail(
+        'Your login link for Superlists',
+        f'Use this link to log in:\n\n{url}',
+        'noreply@superlists',
+        [email],
+    )
+    return render(request, 'login_email_sent.html')
+
+def login(request):
+    print('login view', file=sys.stderr)
+    uid = request.GET.get('uid')
+    user = authenticate(uid=uid)
+    if user is not None:
+        auth_login(request, user)
+    return redirect('/')
+
+def logout(request):
+    auth_logout(request)
+    return redirect('/')
\ No newline at end of file
--- a/functional_tests/test_login.py
+++ b/functional_tests/test_login.py
-# from django.core import mail
-# from selenium.webdriver.common.keys import Keys
-# import re
-
-# from .base import FunctionalTest
-
-# TEST_EMAIL = 'edith@example.com'
-# SUBJECT = 'Your login link for Superlists'
-
-
-# class LoginTest(FunctionalTest):
-
-#     def test_can_get_email_link_to_log_in(self):
-#         # Edith goes to the awesome superlists site
-#         # and notices a "Log in" section in the navbar for the first time
-#         # It's telling her to enter her email address, so she does
-#         self.browser.get(self.live_server_url)
-#         self.browser.find_element_by_name('email').send_keys(TEST_EMAIL)
-#         self.browser.find_element_by_name('email').send_keys(Keys.ENTER)
-
-#         # A message appears telling her an email has been sent
-#         self.wait_for(lambda: self.assertIn(
-#             'Check your email',
-#             self.browser.find_element_by_tag_name('body').text
-#         ))
-
-#         # She checks her email and finds a message
-#         email = mail.outbox[0]
-#         self.assertIn(TEST_EMAIL, email.to)
-#         self.assertEqual(email.subject, SUBJECT)
-
-#         # It has a url link in it
-#         self.assertIn('Use this link to log in', email.body)
-#         url_search = re.search(r'http://.+/.+$', email.body)
-#         if not url_search:
-#             self.fail(f'Could not find url in email body:\n{email.body}')
-#         url = url_search.group(0)
-#         self.assertIn(self.live_server_url, url)
-
-#         # she clicks it
-#         self.browser.get(url)
-
-#         # she is logged in!
-#         self.wait_for(
-#             lambda: self.browser.find_element_by_link_text('Log out')
-#         )
-#         navbar = self.browser.find_element_by_css_selector('.navbar')
-#         self.assertIn(TEST_EMAIL, navbar.text)
--- a/lists/templates/base.html
+++ b/lists/templates/base.html
@@ -16,6 +16,18 @@

    <div class="container">

+      <div class="navbar">
+        {% if user.is_authenticated %}
+          <p>Logged in as {{ user.email }}</p>
+          <p><a id="id_logout" href="{% url 'logout' %}">Log out</a></p>
+        {% else %}
+          <form method="POST" action ="{% url 'send_login_email' %}">
+            Enter email to log in: <input name="email" type="text" />
+            {% csrf_token %}
+          </form>
+        {% endif %}
+      </div>
+
      <div class="row">
        <div class="col-md-6 col-md-offset-3 jumbotron">
          <div class="text-center">

--- a/requirements.txt
+++ b/requirements.txt
@@ -3,4 +3,5 @@ Django==2.1.7
 selenium==3.141.0
 gunicorn==19.9.0
 psycopg2
-dj_database_url
\ No newline at end of file
+dj_database_url
+django_mutpy
\ No newline at end of file
--- a/superlists/settings.py
+++ b/superlists/settings.py
@@ -42,9 +42,16 @@ INSTALLED_APPS = [
    'django.contrib.messages',
    'django.contrib.staticfiles',
    'django_mutpy',
-    'lists'
+    'lists',
+    'accounts',
 ]

+AUTH_USER_MODEL = 'accounts.ListUser'
+AUTHENTICATION_BACKENDS = [
+    'accounts.authentication.PasswordlessAuthenticationBackend',
+]
+
+
 MIDDLEWARE = [
    'django.middleware.security.SecurityMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
@@ -138,3 +145,26 @@ STATIC_URL = '/static/'
 # STATICFILES_DIRS = (
 #     os.path.join(BASE_DIR, 'static'),
 # )
+
+EMAIL_HOST = 'smtp.gmail.com'
+EMAIL_HOST_USER = 'empecempc@gmail.com'
+EMAIL_HOST_PASSWORD = os.environ.get('EMAIL_PASSWORD')
+EMAIL_PORT = 587
+EMAIL_USE_TLS = True
+
+LOGGING = {
+    'version': 1,
+    'disable_existing_loggers': False,
+    'handlers': {
+        'console': {
+            'level': 'DEBUG',
+            'class': 'logging.StreamHandler',
+        },
+    },
+    'loggers': {
+        'django': {
+            'handlers': ['console'],
+        },
+    },
+    'root': {'level': 'INFO'},
+}
\ No newline at end of file
--- a/superlists/urls.py
+++ b/superlists/urls.py
@@ -15,9 +15,12 @@ Including another URLconf
 """
 from django.conf.urls import url, include
 from lists import views, urls as list_urls
+from accounts import urls as accounts_urls
+

 urlpatterns = [
    # url(r'^admin/', admin.site.urls),
    url(r'^$', views.home_page, name='home'),
    url(r'^lists/', include(list_urls)),
+    url(r'^accounts/', include(accounts_urls)),
 ]