Closed Bug 1246735 Opened 8 years ago Closed 8 years ago

http2 - reset recursive push

Categories

(Core :: Networking: HTTP, defect)

Product:
Core
Component:
Networking: HTTP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox48 --- fixed

People

(Reporter: mcmanus, Assigned: mcmanus)

Details

(Whiteboard: [spdy][necko-active])

Attachments

(1 file, 1 obsolete file)

h2 reset recursive push
1.51 KB, patch
u408661
: review+
Details | Diff | Splinter Review 
"PUSH_PROMISE frames MUST only be sent on a peer-initiated stream that is in either the "open" or "half-closed (remote)" state."

we currently allow this
Whiteboard: [spdy]
considering web compat
Assignee: nobody → mcmanus
Whiteboard: [spdy] → [spdy][necko-active]
chrome reports they will reject even associated-id pushes in 51.
From b52ae235cff49c2d5c2846422bf8d4bf7b9a765e Mon Sep 17 00:00:00 2001
---
 netwerk/protocol/http/Http2Session.cpp | 4 ++++
 1 file changed, 4 insertions(+)
Attachment #8728410 - Flags: review?(hurley)

        Attachment #8728411 -
        Flags: review?(hurley)

    
  

        Attachment #8728410 -
        Attachment is obsolete: true

        Attachment #8728410 -
        Flags: review?(hurley)

    
  

        Attachment #8728411 -
        Flags: review?(hurley) → review+

    
  
Keywords: checkin-needed

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/3ca8a82bbff8
Status: NEW → RESOLVED
Closed: 8 years ago

          status-firefox48:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: