[Webkit-unassigned] [Bug 27414] New: Cairo-based Windows port does not handle cookies properly

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Jul 18 07:00:15 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=27414

           Summary: Cairo-based Windows port does not handle cookies
                    properly
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Windows XP
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kwangyul.seo at gmail.com


Cairo-based windows port uses curl as its http backend. curl handles cookies by
itself and it does not share cookies with WinINet. However, CookieJarWin.cpp
uses WinINet functions, InternetGetCookie and InternetSetCookie to get and set
cookies respectively. 

Using CookieJarCurl.cpp is also wrong because it does not interact with curl's
cookie manager. It a simple hash map to get and set cookies. It is inevitable
because there is no such API to get and set cookies in curl.

I think the right direction is to use CookieJarCurl.cpp and ask curl developers
to provide a public API to manipulate cookies. Then we can implement cookie
code correctly.

In the meantime, replacing CookieJarWin.cpp with CookieJarCurl.cpp has no side
effect unless our intention is to share cookies with Internet Explorer or other
WinINet-based clients.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list