If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

RESOLVED FIXED

Status

()

Core
Networking: Cookies
--
trivial
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: AndrewM, Assigned: Robert Millan)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
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.
(Reporter)

Updated

11 years ago
Component: General → Networking: Cookies
Product: Firefox → Core
Version: unspecified → Trunk
QA Contact: general → networking.cookies
(Assignee)

Comment 1

11 years ago
Created attachment 249444 [details] [diff] [review]
fix

I'm attaching a patch.  Can someone set this bug to "confirmed" (or commit the fix ;)?
Thanks for the patch, Robert.
You need to ask review to someone before the patch can be considered for checking in.
Assignee: nobody → rmh
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

11 years ago
Attachment #249444 - Flags: review?(bzbarsky)
Comment on attachment 249444 [details] [diff] [review]
fix

Sure.   Let's do it.
Attachment #249444 - Flags: review?(bzbarsky) → review+
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!
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Comment 5

10 years ago
Is this moot in recent trunk builds?

Seamonkey and Minefield are both using cookies.sqlite.
QA Contact: networking.cookies → benc

Comment 6

10 years ago
yes. (still valid for 1.8 branch but doubt we'll fix it there.)
You need to log in before you can comment on or make changes to this bug.