Unencrypted storage of client side sessions
ImpactThe default configuration of client side sessions results in unencrypted, but signed, data being set as cookie values. This means that if something sensitive goes into the session, it could be read by something with access to the cookies.Note: the documentation does point this out and encourage users to add an encryption key, but it is not mandatory.For this to be a vulnerability, some kind of sensitive data would need to be stored in the session and the session cookie would have to leak. For example, the cookies are not configured with httpOnly and an adjacent XSS vulnerability within the site allowed capture of the cookies.The proposed change is to change the default behaviour to use a randomly generated encryption key. This would mean that sessions do not survive app restarts, but this is already the behaviour given the random signing key.PatchesAs of version 1.9.0, a securely randomly generated signing key is used.WorkaroundsSupply an encryption key, as per the documentation recommendation.Referenceshttps://github.com/ratpack/ratpack/pull/1590