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

Charlie Melbye
Reported 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.
Attachments
[PATCH] Strip port from cookie domains passed into WebInspector.addCookieDomain() (1.52 KB, patch)
2009-12-29 04:35 PST, Alexander Pavlov (apavlov) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Pavel Feldman
Comment 1 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.
Alexander Pavlov (apavlov)
Comment 2 2009-12-29 04:35:28 PST
Created attachment 45591 [details] [PATCH] Strip port from cookie domains passed into WebInspector.addCookieDomain()
WebKit Review Bot
Comment 3 2009-12-29 04:36:32 PST
style-queue ran check-webkit-style on attachment 45591 [details] without any errors.
WebKit Commit Bot
Comment 4 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>
WebKit Commit Bot
Comment 5 2009-12-29 08:51:29 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.