From 5d6ce3671318c8d32bab770ece841590bbec358d 2015-04-17 20:08:32 From: Matthias Bussonnier Date: 2015-04-17 20:08:32 Subject: [PATCH] Set secure cookie by default if login handler is hit. backport of https://github.com/jupyter/jupyter_notebook/pull/22 b8e99bc > There is few chances that logged-in people do not use https connexion, > but I guess it can happened if the server is ran in front of a proxy > that does the https termination, so leave it configurable. > > closes ipython/ipython#8325 --- diff --git a/IPython/html/auth/login.py b/IPython/html/auth/login.py index fa3b6ba..3bfc676 100644 --- a/IPython/html/auth/login.py +++ b/IPython/html/auth/login.py @@ -37,7 +37,13 @@ class LoginHandler(IPythonHandler): typed_password = self.get_argument('password', default=u'') if self.login_available(self.settings): if passwd_check(self.hashed_password, typed_password): - self.set_secure_cookie(self.cookie_name, str(uuid.uuid4())) + # tornado <4.2 have a bug that consider secure==True as soon as + # 'secure' kwarg is passed to set_secure_cookie + if self.settings.get('secure_cookie', self.request.protocol == 'https'): + kwargs = {'secure':True} + else: + kwargs = {} + self.set_secure_cookie(self.cookie_name, str(uuid.uuid4()), **kwargs) else: self._render(message={'error': 'Invalid password'}) return