search

wyona / yanel

http://www.yanel.org
Apache License 2.0
10 stars 5 forks source link

Make Autologin Multi-Cookie aware (Patch from August 2011) #10

Closed baszero closed 8 years ago

baszero commented 12 years ago

This pull request fixes the following: It can happen that a browser stores multiple cookies for the same domain. Yanel's Autologin should support that and consider all cookies sent by the browser.

Information about the pull request: This change has already been sent as SVN Patch back in August 2011 to the Yanel Distribution List already.

--> SO THIS github pull request REPLACES the patch sent in August 2011.

Example: It can happen that the application issues different cookies for different contexts, e.g. Cookie 1: path = "/" Cookie 2: path = "/archive/ Cookie 3: path = "/archive/2011/

In case the user opens the page to archive/2011 directly, the browser sends 3 cookies with the request.

The current Autologin does only process the first cookie, the other two (and maybe the important one) get ignored.

baszero commented 8 years ago

Shall I deliver a new pull request for this patch? Or has this been resolved?

baszero commented 8 years ago

I will resend this patch.