samesite no longer requires bytes

Bert JW Regeer

2018-06-06 6c8ad427ee2f99c30e52667d5a90226de18d2b2d

samesite no longer requires bytes

 pyramid/session.py

@@ -135,7 +135,7 @@
    domain=None,
    secure=False,
    httponly=False,
    samesite=b'Lax',
    samesite='Lax',
    timeout=1200,
    reissue_time=0,
    set_on_exception=True,
@@ -189,7 +189,7 @@
      session cookie. Default: ``False``.

    ``samesite``
      The 'samesite' option of the session cookie. Default ``b'Lax'``.
      The 'samesite' option of the session cookie. Default ``'Lax'``.

    ``timeout``
      A number of seconds of inactivity before a session times out. If
@@ -388,7 +388,7 @@
    cookie_domain=None,
    cookie_secure=False,
    cookie_httponly=False,
    cookie_samesite=b'Lax',
    cookie_samesite='Lax',
    cookie_on_exception=True,
    signed_serialize=signed_serialize,
    signed_deserialize=signed_deserialize,
@@ -442,7 +442,7 @@
      The 'httpOnly' flag of the session cookie.

    ``cookie_samesite``
      The 'samesite' option of the session cookie.  Default: ``b'Lax'``.
      The 'samesite' option of the session cookie.  Default: ``'Lax'``.

    ``cookie_on_exception``
      If ``True``, set a session cookie even if an exception occurs
@@ -502,7 +502,7 @@
    domain=None,
    secure=False,
    httponly=False,
    samesite=b'Lax',
    samesite='Lax',
    set_on_exception=True,
    timeout=1200,
    reissue_time=0,
@@ -566,7 +566,7 @@
      session cookie. Default: ``False``.

    ``samesite``
      The 'samesite' option of the session cookie.  Default: ``b'Lax'``.
      The 'samesite' option of the session cookie.  Default: ``'Lax'``.

    ``timeout``
      A number of seconds of inactivity before a session times out. If

 pyramid/tests/test_session.py

@@ -504,7 +504,7 @@
        expected_cookieval = dummy_signed_serialize(
            (session.accessed, session.created, {'key': 'value'}), secret)
        response = Response()
        response.set_cookie('session', expected_cookieval, samesite=b'Lax')
        response.set_cookie('session', expected_cookieval, samesite='Lax')
        expected_cookie = response.headerlist[-1][1]
        self.assertEqual(cookie, expected_cookie)