Commit Graph

64 Commits

Author SHA1 Message Date
Nicholas Hurley
1f0f26e16a Bug 958712 part 2/3 - Allow CONTINUATIONs in PUSH_PROMISEs r=mcmanus 2015-01-30 11:35:20 -08:00
Nicholas Hurley
d54f25a838 Bug 1127568 - Don't assert in debug builds when receiving CONTINUATION on HEADERS. r=mcmanus 2015-01-30 11:34:01 -08:00
Patrick McManus
e1f56dbdd2 bug 1124717 - 3/4 make h2/spdy default peer max concurrent setting configurable r=hurley 2015-01-26 16:06:44 -05:00
Patrick McManus
c870cb3fa4 bug 1116867 - make nsIProgressEventSink and nsITransportEventSink safely scriptable r=mayhemer r=bz
These scriptable interfaces use uint_64 arguments with sentinel values
of UINT64_MAX. However, UINT64_MAX exceeds MAX_SAFE_INTEGER and cannot
be gatewayed to/from javascript - so they cannot be used
correctly. Change them to use signed 64 bit numbers and -1 as the
sentinnel. C++ implementations ought to be enough to audit as the
special value could never be used correctly in JS anyhow - also
audited OnProgressChange() uses for downstream use of this data.
---
 dom/base/nsXMLHttpRequest.cpp                      | 19 +++++++----
 dom/base/nsXMLHttpRequest.h                        | 10 +++---
 dom/plugins/base/nsPluginStreamListenerPeer.cpp    |  4 +--
 .../webbrowserpersist/nsWebBrowserPersist.cpp      | 14 ++++----
 image/src/imgLoader.cpp                            |  4 +--
 modules/libjar/nsJARChannel.cpp                    |  3 +-
 netwerk/base/public/nsIProgressEventSink.idl       |  8 ++---
 netwerk/base/public/nsITransport.idl               |  8 ++---
 netwerk/base/public/nsNetUtil.h                    | 24 ++++++++++++++
 netwerk/base/src/Dashboard.cpp                     |  2 +-
 netwerk/base/src/nsBaseChannel.cpp                 | 12 +++----
 netwerk/base/src/nsIncrementalDownload.cpp         |  4 +--
 netwerk/base/src/nsSocketTransport2.cpp            |  5 +--
 netwerk/base/src/nsStreamTransportService.cpp      | 38 +++++++++++++---------
 netwerk/base/src/nsTransportUtils.cpp              | 12 +++----
 netwerk/protocol/file/nsFileChannel.cpp            |  8 +++--
 netwerk/protocol/ftp/nsFtpConnectionThread.cpp     |  4 +--
 netwerk/protocol/http/Http2Push.cpp                |  2 +-
 netwerk/protocol/http/Http2Session.cpp             |  2 +-
 netwerk/protocol/http/HttpChannelChild.cpp         | 31 +++++++++---------
 netwerk/protocol/http/HttpChannelChild.h           |  6 ++--
 netwerk/protocol/http/HttpChannelParent.cpp        |  4 +--
 netwerk/protocol/http/HttpChannelParent.h          |  4 +--
 netwerk/protocol/http/NullHttpTransaction.cpp      |  2 +-
 netwerk/protocol/http/PHttpChannel.ipdl            |  2 +-
 netwerk/protocol/http/SpdyPush31.cpp               |  2 +-
 netwerk/protocol/http/SpdySession31.cpp            |  2 +-
 netwerk/protocol/http/TunnelUtils.cpp              |  2 +-
 netwerk/protocol/http/nsAHttpTransaction.h         |  4 +--
 netwerk/protocol/http/nsHttpChannel.cpp            | 30 +++++++++++------
 netwerk/protocol/http/nsHttpConnection.cpp         |  4 +--
 netwerk/protocol/http/nsHttpConnectionMgr.cpp      |  4 +--
 netwerk/protocol/http/nsHttpPipeline.cpp           |  4 +--
 netwerk/protocol/http/nsHttpPipeline.h             |  6 ++--
 netwerk/protocol/http/nsHttpResponseHead.cpp       |  2 +-
 netwerk/protocol/http/nsHttpResponseHead.h         |  2 +-
 netwerk/protocol/http/nsHttpTransaction.cpp        | 32 +++++++++---------
 netwerk/protocol/http/nsHttpTransaction.h          |  2 +-
 netwerk/protocol/wyciwyg/WyciwygChannelChild.cpp   |  2 +-
 netwerk/protocol/wyciwyg/nsWyciwygChannel.cpp      |  3 +-
 netwerk/test/TestIncrementalDownload.cpp           |  7 ++--
 uriloader/base/nsDocLoader.cpp                     | 14 ++++----
 42 files changed, 203 insertions(+), 151 deletions(-)
2015-01-08 14:48:52 -05:00
Patrick McManus
17ab7f34d2 bug 1072478 - h2 push hit not subject to max_concurrent 2/2 r=hurley 2015-01-13 15:26:37 -05:00
Patrick McManus
57d63b6e97 bug 1121706 - don't offer h2 in alpn if w/out mandatory suite r=hurley 2015-01-15 22:11:14 -05:00
Patrick McManus
42a4eb98c2 bug 1121058 - h2 assert stream id before cleanup r=hurley 2015-01-13 13:10:38 -05:00
Patrick McManus
ff1f7611cc bug 1072478 backout 28fff54451dd for 1119810 and 1119926 r=backout 2015-01-09 15:33:27 -05:00
Patrick McManus
2e8dbf08a9 bug 1072478 - h2 push hit not subject to max_concurrent 2/2 r=hurley 2014-10-15 11:51:25 -04:00
Patrick McManus
d16a41862c bug 1003450 - [3/3] Group Dependency Nodes require >= h2-16 r=hurley 2014-12-11 17:00:19 -05:00
Patrick McManus
ad2b489529 bug 1003450 - [2/3] Group Dependency nodes for HTTP/2 r=hurley 2014-12-06 14:26:50 -05:00
Patrick McManus
c96c956a8e bug 1099472 - nspr log argument mismatch h2::recvpushpromise r=hurley 2014-11-14 20:34:40 -05:00
Honza Bambas
0cf0bcaa7a Bug 1081089 - Move spdy io logs to level 5, r=mcmanus 2014-11-10 16:10:41 +01:00
Nicholas Hurley
dc77764414 Bug 1049814 - Retry REFUSED_STREAMs on the same session. r=mcmanus 2014-11-06 09:47:00 -05:00
Nicholas Hurley
14b47a8942 Bug 1091263 - HTTP_1_1_REQUIRED support. r=mcmanus 2014-11-06 09:26:00 -05:00
Nicholas Hurley
b8f37776aa Bug 1055306 - Make h2 push origin check more correct. r=mcmanus 2014-11-03 16:19:00 +01:00
Patrick McManus
45b91c0db1 bug 1024730 - nsIHttpPushListener r=hurley
co-author: ben brittain <ben@brittain.org>
2014-10-21 14:35:41 -04:00
Patrick McManus
eed5fe2837 bug 1073825 - http2session::cleanupstream failure r=hurley 2014-10-13 11:02:08 -04:00
Patrick McManus
590bd04d86 bug 1081341 - h2 1xx responses r=hurley 2014-09-27 09:06:38 -04:00
Patrick McManus
30e70bb8f9 bug 1003448 - HTTP/2 Alternate Service and Opportunistic Security [2/2 necko] r=hurley 2014-08-21 10:50:17 -04:00
Daniel Stenberg
5b4c276b42 Bug 939318 - Find and close HTTP connections without traffic after network change. r=mcmanus
SPDY/http2 connections get a ping and be allowed N seconds to respond.

Active HTTP connections will be allowed N seconds to get traffic, if they
don't afer N seconds they get closed to avoid risking stalled transfers.

N is 5 by default: pref is "network.http.network-changed.timeout"
2014-09-11 06:55:00 -04:00
Ed Morley
cc6bc5ce84 Backed out changeset 0a066b7ffa46 (bug 939318) 2014-09-24 16:17:32 +01:00
Daniel Stenberg
9ac9528ccf Bug 939318 - Find and close HTTP connections without traffic after network change, r=mcmanus
SPDY/http2 connections get a ping and be allowed N seconds to respond.

Active HTTP connections will be allowed N seconds to get traffic, if they
don't afer N seconds they get closed to avoid risking stalled transfers.

N is 5 by default: pref is "network.http.network-changed.timeout"
2014-09-11 06:55:00 -04:00
Nicholas Hurley
7ba4697ffa Bug 1046915 - Pref to advertise the latest HTTP/2 draft as "h2". r=mcmanus 2014-09-23 10:49:13 -07:00
Tom Tromey
a050e72780 Bug 1069387 - Fix "Transaction" typos in netwerk/protocol/http. r=mcmanus 2014-09-18 08:13:00 -04:00
Patrick McManus
be8b2fae1d bug 1050063 - consider tls client hello version in alpn/npn offer list r=hurley r=keeler 2014-08-15 09:39:53 -04:00
Nicholas Hurley
f3d5f209d2 Bug 1054173 - Do not fail connections when receiving request headers. r=mcmanus 2014-08-18 14:59:32 -07:00
Nicholas Hurley
9f0dc48e2b Bug 1046892 part 1 - HTTP/2 draft 14 client implementation r=mcmanus 2014-08-05 08:41:09 -07:00
Patrick McManus
940032ac8c bug 1027720 - enforce h2 requirement that sever uses aead r=hurley r=dkeeler 2014-08-04 16:54:10 -04:00
Patrick McManus
3b584604e6 bug 1037082 - part 3/3 407 proxy auth for https over https r=hurley 2014-07-07 11:43:11 -04:00
Nicholas Hurley
e896b85c44 Bug 1014600 part 2 - Properly truncate buffers when processing new header blocks. r=mcmanus 2014-07-11 09:48:34 -07:00
Nicholas Hurley
1d2a554653 Bug 1030203 - Send PROTOCOL_ERROR when receiving PUSH_PROMISE with push disabled. r=mcmanus 2014-06-27 13:13:42 -07:00
Nicholas Hurley
c4ea7ca39f Bug 1026609 - HTTP/2 draft 13 part 1 - client implementation. r=mcmanus 2014-06-25 17:33:15 -07:00
Patrick McManus
cdd3e5dfb8 bug 1027364 - multiple http/2 header frames on same stream r=hurley 2014-06-19 23:42:29 -04:00
Daniel Stenberg
cf9d1ef515 Bug 1027353: avoid Http2Session NULL deref. r=hurley 2014-06-18 23:15:00 -04:00
Daniel Stenberg
896e58f7b4 Bug 237623 - detect broken HTTP1.1 transfers. r=mcmanus,seth
Return error when the protocol layer detects a framing error. More data
was supposed to be delivered than what actually did arrive.

Error code returned for this: NS_ERROR_NET_PARTIAL_TRANSFER

In HTTP1.1 for Content-Length: and chunked-encoding underruns

In http2 and SPDY for framing errors when data has already been received.

imgRequest::OnStopRequest will keep partially loaded images shown but
remove them from cache.
2014-06-09 00:15:00 +02:00
Patrick McManus
d49cece06f bug 1025075 - http2 CONNECT streams should have consistent priority r=hurley
---
 netwerk/protocol/http/Http2Session.cpp  | 3 ++-
 netwerk/protocol/http/SpdySession3.cpp  | 3 ++-
 netwerk/protocol/http/SpdySession31.cpp | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

---
 netwerk/protocol/http/Http2Session.cpp  | 3 ++-
 netwerk/protocol/http/SpdySession3.cpp  | 3 ++-
 netwerk/protocol/http/SpdySession31.cpp | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)
2014-06-13 09:29:00 -04:00
Nicholas Hurley
dc4de7d7c4 Bug 1019918 - Remove pushed stream from SpdyPushCache when cleaning it up. r=mcmanus 2014-06-04 15:37:00 -04:00
Nicholas Hurley
aa06db2ea3 Bug 1019921 - [http/2] Don't change to discarding padding state when all padding has been processed. r=mcmanus 2014-06-03 19:41:26 -07:00
Patrick McManus
9c03366117 bug 1012825 - https proxying over http/2 connect stream r=hurley 2014-05-19 15:37:35 -04:00
Patrick McManus
60f9707b98 bug 378637 part 14 - https proxying for spdy31 and http2 r=hurley 2014-05-16 11:46:13 -04:00
Patrick McManus
c42552a198 bug 378637 part 12 - proxy over TLS (i.e. https proxying) r=hurley 2014-04-16 09:52:43 -04:00
Patrick McManus
233a352113 bug 378637 part 10 - implement nsAHttpTransaction::ConnectionInfo r=hurley 2014-04-20 12:01:35 -04:00
Patrick McManus
d59917f107 bug 378637 part 7 - new spdysession() no longer takes first transaction r=hurley 2014-05-16 11:46:11 -04:00
Patrick McManus
cbec6a3a17 bug 378637 part 5 - move Spdy*::EnsureBuffer to nsHttp r=hurley 2014-04-15 17:06:59 -04:00
Patrick McManus
e4716c63eb bug 378637 part 3 - fix http style: comments, whitespace, formatters r=hurley 2014-04-17 19:55:04 -04:00
Nicholas Hurley
090834c415 Bug 1006804 - Enforce TLS key type and size restrictions in HTTP/2. r=mcmanus. 2014-05-14 10:37:05 -07:00
Daniel Stenberg
0c35f6fa0d Bug 1007024 - CopyAsNetwork32 inserts 32bit network byte order values. r=mcmanus
CopyAsNetwork32 is a convenience function to make the PR_htonl() +
memcpy use in the code less repetitive.
2014-05-14 01:01:00 +02:00
Ryan VanderMeulen
9ee7c1593b Backed out 11 changesets (bug 378637) for Android crashes.
Backed out changeset 2a607cddc4cb (bug 378637)
Backed out changeset e416503aea99 (bug 378637)
Backed out changeset b2257226899f (bug 378637)
Backed out changeset dafd618c3f52 (bug 378637)
Backed out changeset dfde9d47d8c4 (bug 378637)
Backed out changeset cf9de5c367a5 (bug 378637)
Backed out changeset 62aa68e8b499 (bug 378637)
Backed out changeset 38efa8f2e56e (bug 378637)
Backed out changeset 2b5753e09a92 (bug 378637)
Backed out changeset 7a73873e133d (bug 378637)
Backed out changeset f58ce7ac1c7f (bug 378637)
2014-05-09 15:25:55 -04:00
Patrick McManus
7c5417cb13 bug 378637 part 11 - proxy over TLS (i.e. https proxying) r=hurley 2014-04-16 09:52:43 -04:00