search

benlucchesi / grails-cookie-session

cookie sessions for grails applications
28 stars 32 forks source link

Don't store blank session cookies #26

Closed jrh3k5 closed 10 years ago

jrh3k5 commented 10 years ago

The CookieSessionRepository will store a session cookie, even if it's blank. While this isn't a terrible cost, it does mean unnecessary headers and cookies in each response. It'd be preferable for it to skip the storage of a cookie if the value is blank.

benlucchesi commented 10 years ago

this feature was implemented in 2.0.13. If you find that the plugin isn't behaving as described, please reopen this issue.

Thanks!