Closed
Bug 581905
Opened 14 years ago
Closed 14 years ago
[stats] move IP address to variable
Categories
(Camino Graveyard :: General, enhancement)
Camino Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ss, Assigned: ss)
References
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Updated•14 years ago
|
Attachment #460211 -
Flags: review?(alqahira)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → samuel.sidler
Comment on attachment 460211 [details]
Patch v1
I assume the $ inside the quotes is a typo; r=me without it. If you were trying to do something with it, I'm not sure what it was (let me know!), since it prevents matching that first bad IP.
(I tested this against July 19 and July 21; with the stray $ removed, data comes out the same pre- and post-patch.)
Attachment #460211 -
Flags: review?(alqahira) → review+
Blocks: 582113
Assignee | ||
Comment 2•14 years ago
|
||
(In reply to comment #1)
> I assume the $ inside the quotes is a typo; r=me without it.
Indeed. And I have it removed in my local (well, remote) copy.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•