search

TyMaszWeb / django-cookie-law

Helps your Django project comply with the EU cookie regulations by displaying a cookie information banner until it is dismissed by the user
BSD 2-Clause "Simplified" License
140 stars 69 forks source link

Better handling of missing context 'request' attribute #57

Closed aragilar closed 4 years ago

aragilar commented 4 years ago

As noted in https://github.com/TyMaszWeb/django-cookie-law/issues/9#issuecomment-127992714, the check for context.request falls though and hence a KeyError is still thrown. This PR contains a fix which stops the falling though that wasn't merged previously, which will fix #9 (which is incorrectly closed).

piotrkilczuk commented 4 years ago

Released as 2.0.3. Thanks for this.