- 10 Aug, 2016 1 commit
-
-
Tatsuhiro Tsujikawa authored
-
- 09 Aug, 2016 3 commits
-
-
Tatsuhiro Tsujikawa authored
-
-
Wenfeng Liu authored
-
- 07 Aug, 2016 2 commits
-
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
Previously, in server side, we used closed streams to detect the error that the misbehaving client sends a frame on the incoming stream it explicitly closed. With this commit, we make a further step, and detect one more error case. Since we retain closed streams as long as the sum of its size and the number of opened streams are equal or less than max concurrent streams, we can safely say that if we get a frame which is sent on the stream that is not found in either closed or opened stream, it is already closed or has not existed. Then we can send GOAWAY. The previous code shrinks closed streams when we closed another stream, but now it is removed. It is enough to adjust closed streams when new incoming stream is created. While creating this commit, we noticed that NGHTTP2_INITIAL_MAX_CONCURRENT_STREAMS is defined as INT32_MAX. But since SETTINGS can contain value up to UINT32_MAX, it is not enough. However, since the stream ID space is limited to INT32_MAX, it is high enough. We could keep this value, but this time we deprecate NGHTTP2_INITIAL_MAX_CONCURRENT_STREAMS macro. While it is in public header, the effect of deprecating it is negligible because of the reason we wrote above, and usually application sets much smaller value (say, 100) as SETTINGS_MAX_CONCURRENT_STREAMS.
-
- 06 Aug, 2016 2 commits
-
-
Tatsuhiro Tsujikawa authored
-
-
- 05 Aug, 2016 1 commit
-
-
Wenfeng Liu authored
2. Remove inline qualifier of lookup_token() in genlibtokenlookup.py.
-
- 04 Aug, 2016 6 commits
-
-
Tatsuhiro Tsujikawa authored
Use $backend_host and $backend_port. $backend_host is backend host name given in --backend option. It could be a path to UNIX domain socket.
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
-
Tatsuhiro Tsujikawa authored
https://tools.ietf.org/html/rfc7540#section-5.3.3 explains how to transform dependency tree to avoid circular dependency. Previously, we wrongly always moved the dependent stream under the root stream. The correct destination is the parent stream of the stream to reprioritize. This commit fixes this bug.
-
Wenfeng Liu authored
to do extra calculation
-
- 02 Aug, 2016 3 commits
-
-
Tatsuhiro Tsujikawa authored
Previously, for PUSH_PROMISE frame, we logged its promised stream ID. But it is saner to use just frame's ID in this context.
-
Tatsuhiro Tsujikawa authored
We have a code to call error callback when invalid header is received and it is treated as stream error. But we didn't if the incoming header is invalid, but just ignored. This generosity is required to handle public Internet connections especially when nghttp2 is used as forward proxy.
-
Tatsuhiro Tsujikawa authored
This reverts commit f4a4abd1.
-
- 31 Jul, 2016 12 commits
-
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
Now config global is backed with std::unique_ptr. configuration swapping dance is now a bit cleaner, but YMMV.
-
Tatsuhiro Tsujikawa authored
This commit also fixes the bug that old configuration is still used for worker process. The another bug fix is that inherited, but not used fd is not closed in worker process. That makes reloading next configuration fail if it contains the address which are leaked into worker process.
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
The last_worker_pid is known by inspecting the last entry of worker_processes.
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
This commit implements configuration reloading with SIGHUP. There are rough edges left: * Rename SignalServer with more meaningful name, say, WorkerProcess. * We should introduce global configuration object which is not affected by configuration reloading. It should hold cmdcfgs, argc, argv, and last worker PID. * We should close the listener file descriptor when some operation was failed after that.
-
- 30 Jul, 2016 1 commit
-
-
Tatsuhiro Tsujikawa authored
-
- 29 Jul, 2016 3 commits
-
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
- 26 Jul, 2016 1 commit
-
-
Tatsuhiro Tsujikawa authored
The N-th -p options sets N-th URI's weight. If the number of -p is less than the number of URIs, the last -p value is repeated. If no -p is set, use default weight, 16.
-
- 21 Jul, 2016 4 commits
-
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
Tatsuhiro Tsujikawa authored
-
- 17 Jul, 2016 1 commit
-
-
Tatsuhiro Tsujikawa authored
-