Skip to content

Conversation

@honsiorovskyi
Copy link

@honsiorovskyi honsiorovskyi commented Jul 6, 2016

Added 127.0.0.0/8 IP range to the expression which checks whether the session cookie domain is 'localhost' in the method get_cookie_domain.


This change is Reviewable

Added 127.0.0.0/8 IP range to the expression which checks whether the session cookie domain is 'localhost' in the method `get_cookie_domain`.
@untitaker
Copy link
Contributor

untitaker commented Sep 6, 2016

I feel like this is an incomplete fix for related to #1906

@untitaker
Copy link
Contributor

untitaker commented Sep 6, 2016

Actually, SERVER_NAME does not support IP addresses.

EDIT: Here's the spec: https://tools.ietf.org/html/rfc6265#section-5.1.3

@untitaker untitaker closed this Sep 6, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants