Merge pull request #200 from opdemand/198-user-isnt-active

Fixed #198 -- ensure new users have is_active == True.

Author: Gabriel Monroy

api/tests/auth.py

@@ -48,6 +48,7 @@ def test_auth(self):
         self.assertEqual(response.data['email'], email)
         self.assertEqual(response.data['first_name'], first_name)
         self.assertEqual(response.data['last_name'], last_name)
+        self.assertTrue(response.data['is_active'])
         self.assertFalse(response.data['is_superuser'])
         self.assertFalse(response.data['is_staff'])
         self.assertTrue(

api/views.py

@@ -78,6 +78,7 @@ def pre_save(self, obj):
         now = timezone.now()
         obj.last_login = now
         obj.date_joined = now
+        obj.is_active = True
         obj.email = User.objects.normalize_email(obj.email)
         obj.set_password(obj.password)
 

client/deis.py

@@ -576,14 +576,16 @@ def auth_register(self, args):
             password = getpass('password: ')
             confirm = getpass('password (confirm): ')
             if password != confirm:
-                print('Password mispatch, aborting registration.')
+                print('Password mismatch, aborting registration.')
                 return False
         email = args.get('--email')
         if not email:
             email = raw_input('email: ')
         url = urlparse.urljoin(controller, '/api/auth/register')
         payload = {'username': username, 'password': password, 'email': email}
-        response = self._session.post(url, data=payload, allow_redirects=False)
+        response = self._session.post(url, data=payload,
+                                      headers={'content-type': 'application/json'},
+                                      allow_redirects=False)
         if response.status_code == requests.codes.created:  # @UndefinedVariable
             self._settings['controller'] = controller
             self._settings.save()