Fixed #20444 -- Cookie-based sessions does not include a remote code execution warning #1119

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@mxsasha
Member

mxsasha commented May 18, 2013

No description provided.

@aaugustin

This comment has been minimized.

Show comment Hide comment
@aaugustin

aaugustin May 18, 2013

Owner

Pushed after adding a missing colon in d5ce2ff.

Owner

aaugustin commented May 18, 2013

Pushed after adding a missing colon in d5ce2ff.

@aaugustin aaugustin closed this May 18, 2013

@mxsasha mxsasha deleted the mxsasha:better-cookie-warnings branch May 18, 2013

@mxsasha mxsasha restored the mxsasha:better-cookie-warnings branch May 18, 2013

@mxsasha

This comment has been minimized.

Show comment Hide comment
@mxsasha

mxsasha May 18, 2013

Member

@aaugustin I'm not sure why it does not work, but there seems to be something wrong with the first link in the patch for SECRET_KEY (which I think you added): look at the current https://docs.djangoproject.com/en/dev/topics/http/sessions/#using-cookie-based-sessions

Member

mxsasha commented May 18, 2013

@aaugustin I'm not sure why it does not work, but there seems to be something wrong with the first link in the patch for SECRET_KEY (which I think you added): look at the current https://docs.djangoproject.com/en/dev/topics/http/sessions/#using-cookie-based-sessions

@aaugustin

This comment has been minimized.

Show comment Hide comment
@aaugustin

aaugustin May 19, 2013

Owner

Oops, fixed in 753edfa .

Owner

aaugustin commented May 19, 2013

Oops, fixed in 753edfa .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment