Comment #2 on issue 806 by [email protected]:
ContinuingHttpServletRequest: Make cookies immutable
http://code.google.com/p/google-guice/issues/detail?id=806
Thanks. The provided patch is missing null pointer check in constructor:
if (super.getCookies() != null) {
cookies = super.getCookies().clone();
}
In our use case the cookies exist only when ContinuingHttpServletRequest
object is created. Later in background thread no cookies available any more
on the underlying wrapper HTTP request.
Cloning and freezing the cookies in constructor of
ContinuingHttpServletRequest solves that.
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
You received this message because you are subscribed to the Google Groups
"google-guice-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/google-guice-dev.
For more options, visit https://groups.google.com/d/optout.
