mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #22135
[Bug 1384009] Re: Cookie lacking "secure" flag for HTTPS sites
I just uploaded some patches for this... but then I noticed that Robert
already uploaded some patches as well!
Here's mine:
https://reviews.mahara.org/3938
https://reviews.mahara.org/3939
Here's Robert's:
https://reviews.mahara.org/#/c/3898
https://reviews.mahara.org/#/c/3899
** Information type changed from Private Security to Public Security
** Changed in: mahara/1.10
Importance: Undecided => Low
** Changed in: mahara/1.8
Importance: Undecided => Low
** Changed in: mahara/1.9
Importance: Undecided => Low
--
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:
In Progress
Status in Mahara 1.10 series:
New
Status in Mahara 1.8 series:
New
Status in Mahara 1.9 series:
New
Status in Mahara 15.04 series:
In Progress
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