Closed Bug 17685 Opened 25 years ago Closed 25 years ago

Need a way to save post data in Session History and repost on B/F

Categories

(Core :: Networking, defect, P1)

Product:
Core
Component:
Networking
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: radha, Assigned: radha)

References

Details

(Whiteboard: [nsbeta2+][PDT-]) 

When accessing a page with post data and going Back/Forward to it, session
History needs to save the post data. Though SH *will soon* have the ability to
save a handle to post data(nsIInputStream) in the cache, it will not be able to
reset the stream to the beginning and repost, because nsIInputStream doesn't
support that feature.

Because of this going Back/Forward to pages that use post data like my.yahoo.com
login screen or travelocity login screen doesn't behave right.
Priority: P3 → P1
Assignee: gagan → warren
Status: NEW → ASSIGNED
Target Milestone: M11
I'll take this one.
Assignee: warren → radha
Status: ASSIGNED → NEW
I've checked in the code to make the stream returned from NS_NewPostDataStream
support the nsIRandomAccessStore interface. So now the task is to save the
stream in the history, and before reposting, Seek back to zero.

Off to Radha.
Status: NEW → ASSIGNED
*** Bug 3805 has been marked as a duplicate of this bug. ***
*** Bug 16782 has been marked as a duplicate of this bug. ***
Blocks: 18471
*** Bug 20154 has been marked as a duplicate of this bug. ***
Bulk move of all Necko (to be deleted component) bugs to new Networking

component.
*** Bug 21954 has been marked as a duplicate of this bug. ***
Summary: Need away to save post data in Session History and repost on B/F → [4.xP]Need away to save post data in Session History and repost on B/F
You shouldn't actually repost the data without asking the user.  That could
cause people to do things like buy something twice, etc.  The 4.x behavior here
is good.
Target Milestone: M13 → M14
Moving all to M14 (for webshell changes that are being deferred till M14 rather
than destabilize M13).
Setting the keyword all open [4.xp] bugs to 4xp.
Keywords: 4xp
Marking as beta1.
Keywords: beta1
Putting on PDT+ radar for beta1.
Summary: [4.xP]Need away to save post data in Session History and repost on B/F → [4.xP]Need a way to save post data in Session History and repost on B/F
Whiteboard: [PDT+]
Summary: [4.xP]Need a way to save post data in Session History and repost on B/F → Need a way to save post data in Session History and repost on B/F
*** Bug 26208 has been marked as a duplicate of this bug. ***
*** Bug 27127 has been marked as a duplicate of this bug. ***
Bill sez this should "just work" when we flip the switch on the new session
history stuff.
Depends on: 13374
I just tried this and it works for me (going off of don's last statement on the
10th of February).  Marking fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Testing in Linux mozilla 2000-02-28-08-M15.

verified:
WinNT 3/05/00 
Linux 2000030609
Mac 2000030508
Status: RESOLVED → VERIFIED
Unless there is another bug report out there for sites *without* frames, this is
still broken in Win98 build 2000030708.  Here is a sample:

1. Turn off memory cache (otherwise CSS and such won't load at this site)
2. Go to http://www.vortxweb.net/ccs-pg/my-cgi/toykar/chayka5.cgi
3. Make a selection from the list boxes
4. Submit ("get prices")
5. Check any of the boxes
6. Submit ("buy")
7. Press the browser back button.
8. The page of check box selections is wiped out

However, if you go back to the first page from the second or third page, the
original selections are still highlighted, so only the first page was saved
intact.  Post data from subsequent pages is toast.


I also can't verify this - looks like old session history is still being used,
and it doesn't store/reset/repost the post data stream, reopening.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Blocks: 27006
I'm removing the PDT+ to get a re-evaluation (it was marked plus several weeks
ago, and resolved, only to be re-opened today).  We are probably too close to a
beat 1 ship to take a major change in this area.  PDT will comment later today.
Whiteboard: [PDT+]
Moving to M15...
Target Milestone: M14 → M15
Putting on PDT- radar for beta1.
Whiteboard: [PDT-]
*** Bug 32787 has been marked as a duplicate of this bug. ***
Nominating for beta2
Keywords: beta2
Whiteboard: [PDT-]
Putting on the PDT- radar for beta1.
Whiteboard: [PDT-]
Status: REOPENED → ASSIGNED
Move to M16 for now ...
Target Milestone: M15 → M16
Keywords: nsbeta2
*** Bug 38697 has been marked as a duplicate of this bug. ***
*** Bug 38732 has been marked as a duplicate of this bug. ***
Putting on [nsbeta2+] radar for beta2 fix.    claudius, please retest or find 
dup.
URL: [nsbeta2+]
QA Contact: tever → claudius
I'm gonna take a wild shot in the dark here and assume you didn't mean to put 
the nsbeta2+ marking in the URL field.  Moving to whiteboard (I assume the PDT- 
marking should be removed, but I'll let someone else do that...)
URL: [nsbeta2+]
Whiteboard: [PDT-] → [nsbeta2+][PDT-]
radha, isn't this the same issue or root cause as bug 1718?

Yes it is. I'm hanging on to both so that when I fix, I can test 1718.
Depends on: 39668
Move to M18.
Target Milestone: M16 → M18
Fix in hand. Shall checkin once tree opens
Target Milestone: M18 → M16
checked in fix.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Target Milestone: M16 → M18
can someone please verfiy this, for the release notes. thank you very much.
something happened to the resolution on this one (bug 3273)

re-marking fixed, sorry everybody for the spam
Status: RESOLVED → REOPENED
fixed
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
No longer blocks: 18471
VERIFIED Fixed with 2000070708 builds
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.