Send Page needs to be disabled for pages with post data

VERIFIED FIXED

Status

SeaMonkey
MailNews: Message Display
VERIFIED FIXED
17 years ago
14 years ago

People

(Reporter: vidur (gone), Assigned: Blake Ross)

Tracking

Trunk
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: PDT+ fix in 91341)

(Reporter)

Description

17 years ago
Send Page, when invoked for a page with post data (generally not cached),
results in a HTTP GET request. The fix for this is being discussed in bug 40867.
In lieu of the complete fix, we may need to disable send page for pages with
post data on the branch.
(Reporter)

Updated

17 years ago
Depends on: 40867
(Reporter)

Updated

17 years ago
No longer depends on: 40867

Comment 1

17 years ago
We'll take this, we need something immediately. Please get the patch & reviews
and checkin ASAP.
Whiteboard: PDT+
-> Blake
-> blake
Assignee: chofmann → blake
How is blake to test the patch ? We need to know a page where this happens. Is 
this only for pages with post data that cannot be cached (are there any common 
examples of that)?

Comment 5

17 years ago
cc: putterman as a fyi as this has to do with send page.

vishy - I'm not sure of a page on how to test this other than a confirmation
page which comes up after a transaction.  Can you email the web platform folks
in Clayton's group?  They may have an idea.

esther - will need to release note this.
(Assignee)

Comment 6

17 years ago
The fix for this is in bug 91341; there, I also gave a good page to test on.
Whiteboard: PDT+ → PDT+ fix in 91341
(Assignee)

Comment 7

17 years ago
cc'ing scott
(Assignee)

Comment 8

17 years ago
fix is in.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
Using the attachment in bug 91341 to submit, Send Page is disabled on the
resulting data page.
OK using july19 commercial 0.9.2 branch builds: win98, linux rh6.2 and mac OS 9.0

Updated

17 years ago
Status: RESOLVED → VERIFIED

Comment 10

17 years ago
Since this is branch only, temporary fix to disable in lieu of "real" fix for
the underlying problem, I'm going to mark this verified.  Added this bug number
to release note tracking bug 90577
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.