Permalink
Browse files

Merge pull request #19 from authorNari/fix-assoc-id

SynStream doesn't set an associated_to_stream_id.
  • Loading branch information...
2 parents e9857e9 + 2c7d04c commit 91f61f4e0f11b562b075c24003c2d00dfa7e3bab @igrigorik committed Jan 25, 2013
Showing with 3 additions and 1 deletion.
  1. +1 −0 lib/spdy/protocol.rb
  2. +2 −1 spec/protocol_spec.rb
View
1 lib/spdy/protocol.rb
@@ -83,6 +83,7 @@ class SynStream < BinData::Record
def create(opts = {})
build({:type => 1, :len => 10}.merge(opts))
+ self.associated_to_stream_id = opts.fetch(:associated_to_stream_id, 0)
end
end
View
3 spec/protocol_spec.rb
@@ -46,9 +46,10 @@
"url"=>"/?echo=a&format=json","version"=>"HTTP/1.1"
}
- sr.create({:stream_id => 1, :headers => headers})
+ sr.create({:stream_id => 1, :headers => headers, :associated_to_stream_id => 2})
sr.header.version.should == 2
sr.pri.should == 0
+ sr.associated_to_stream_id.should == 2
sr.header.len.should > 50
sr.data.should_not be_nil

0 comments on commit 91f61f4

Please sign in to comment.