Bug 31787

Summary: Cookies don't show up in Web Inspector storage tab if website has a port
Product: WebKit Reporter: Charlie Melbye <charlie>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, commit-queue, dbates, joepeck, keishi, pfeldman, pmuellr, rik, timothy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://grab.by/Jyn
Attachments:
Description Flags
[PATCH] Strip port from cookie domains passed into WebInspector.addCookieDomain() none

Description Charlie Melbye 2009-11-22 10:45:01 PST 
Cookies for a website are not showing up correctly in Web Inspector's storage tab if the website has a port. I noticed this while developing my Rails application locally at "localhost:3000". Under the "Cookies" section, it says "localhost:3000". I don't think cookies are stored on a per-port basis, so the port should be stripped off the web address before looking for the cookies.
Comment 1 Pavel Feldman 2009-12-04 12:02:28 PST 
Before one jumps into fixing this, note that there is a small cookies refactoring in https://bugs.webkit.org/show_bug.cgi?id=32160 under review.
Comment 2 Alexander Pavlov (apavlov) 2009-12-29 04:35:28 PST 
Created attachment 45591 [details]
[PATCH] Strip port from cookie domains passed into WebInspector.addCookieDomain()
Comment 3 WebKit Review Bot 2009-12-29 04:36:32 PST 
style-queue ran check-webkit-style on attachment 45591 [details] without any errors.
Comment 4 WebKit Commit Bot 2009-12-29 08:51:23 PST 
Comment on attachment 45591 [details]
[PATCH] Strip port from cookie domains passed into WebInspector.addCookieDomain()

Clearing flags on attachment: 45591

Committed r52623: <http://trac.webkit.org/changeset/52623>
Comment 5 WebKit Commit Bot 2009-12-29 08:51:29 PST 
All reviewed patches have been landed.  Closing bug.