mahara-contributors team mailing list archive

[Aaron Wells <1384009@xxxxxxxxxxxxxxxxxx>]

Because Cookie Consent doesn't have a public bug tracker anywhere, I
sent them an email to share the patch https://reviews.mahara.org/3939
back upstream.

Although I suppose for general-purpose Cookie Consent use, they probably
won't want to upstream that patch. Since Cookie Consent may be used on
mixed-content sites, in which it would actually be preferrably *not* to
have the "secure" flag set.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1384009

Title:
  Cookie lacking "secure" flag for HTTPS sites

Status in Mahara ePortfolio:
  Fix Committed
Status in Mahara 1.10 series:
  Fix Committed
Status in Mahara 1.8 series:
  Fix Committed
Status in Mahara 1.9 series:
  Fix Committed
Status in Mahara 15.04 series:
  Fix Committed

Bug description:
  The cookie "lastinstitution" that we use to show the proper
  institution theme to logged-out users, does not properly use the
  "secure" attribute for sites that are using HTTPS. This means it's
  possible for the cookie's contents to be obtained via non-HTTPS.

  Not a huge thing, since its use is somewhat limited in scope, and the
  "lastinstitution" data is not very sensitive, but it would be good to
  use it.

  While we're at it, we might also want to check on the (much more
  important) PHP session cookie. This can be set at the server level,
  but we could also check for it in PHP. See
  http://stackoverflow.com/questions/6821883/set-httponly-and-secure-on-
  phpsessid-cookie-in-php for details on that.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1384009/+subscriptions