346125 - Broken link to Netscape Cookie Specification page in cookies.txt file

Status: RESOLVED FIXED

(Core :: Networking: Cookies, defect)

Description

[AndrewM]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060726 Minefield/3.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060726 Minefield/3.0a1

In the automatically-generated cookies.txt file in the profile directory, the link to the "Persistent Client State HTTP Cookies - Preliminary Specification" page is broken. The link currently points to <http://www.netscape.com/newsref/std/cookie_spec.html>; however it should point to <http://wp.netscape.com/newsref/std/cookie_spec.html>.

Reproducible: Always

Steps to Reproduce:
1. Open up the cookies.txt file.
2. Open the URL at the beginning of the file in Firefox or any other web browser.
Actual Results:  
Not Found

The requested URL /newsref/std/cookie_spec.html was not found on this server.

Expected Results:  
The specification page opens.

Comment 1

[Robert Millan]

Attachment: fix

I'm attaching a patch.  Can someone set this bug to "confirmed" (or commit the fix ;)?

Comment 2

[Martijn Wargers (dead)]

Thanks for the patch, Robert.
You need to ask review to someone before the patch can be considered for checking in.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 249444 [details] [diff] [review]
fix

Sure.   Let's do it.

Comment 4

[Martijn Wargers (dead)]

Checking in nsCookieService.cpp;
/cvsroot/mozilla/netwerk/cookie/src/nsCookieService.cpp,v  <--  nsCookieService.
cpp
new revision: 1.49; previous revision: 1.48
done

Checked into trunk.
Again, thanks for the patch, Robert!

Comment 5

[benc]

Is this moot in recent trunk builds?

Seamonkey and Minefield are both using cookies.sqlite.

Comment 6

[[e-mail]]

yes. (still valid for 1.8 branch but doubt we'll fix it there.)