Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/763
@zwoop Looks good. We should land this to add a option to get metrics per
protocol.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/790
Looks good to me
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/593
I updated PO files in 0ebfab01d909bcd0d99cd2f8fb2f816916bc80f3.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/733
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/767
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/767
TS-4605: fix Coverity issue 1357058
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/masaori335/trafficserver ts-4605
Alternatively
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/760
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/760#discussion_r68878648
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -1054,7 +1055,7 @@ Http2ConnectionState::send_a_data_frame(Http2Stream
*stream, size_t
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/760#discussion_r68876890
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -1054,7 +1055,7 @@ Http2ConnectionState::send_a_data_frame(Http2Stream
*stream, size_t
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/593
@mingzym I'll update Japanese docs after this is merged if you want :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/749
Do we need some changes in `HttpTransact::build_response()` too?
https://github.com/apache/trafficserver/blob/master/proxy/http/HttpTransact.cc#L7945-L7975.
---
If your project is
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/749#discussion_r68698335
--- Diff: proxy/http/HttpTransact.cc ---
@@ -8199,13 +8199,15 @@ HttpTransact::build_error_response(State *s,
HTTPStatus status_code, const char
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/733#discussion_r68175695
--- Diff: proxy/http2/Http2ClientSession.h ---
@@ -94,63 +89,60 @@ class Http2Frame
{
this->ioblock = new_IOBufferBl
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/733
TS-4324: Allocate 16KB for DATA frame payload exactly
- [TS-4324
comment-15313205](https://issues.apache.org/jira/browse/TS-4324?focusedCommentId=15313205&
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/708
Looks good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/708#discussion_r67087190
--- Diff: lib/ts/PriorityQueue.h ---
@@ -115,6 +117,22 @@ PriorityQueue::pop()
template
void
+PriorityQueue::erase
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/686
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/686
FreeBSD build successful
https://ci.trafficserver.apache.org/job/Github-FreeBSD/189/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/686#discussion_r65658324
--- Diff: mgmt/api/INKMgmtAPI.cc ---
@@ -1865,8 +1865,7 @@ TSGetErrorMessage(TSMgmtError err_id)
tsapi TSMgmtError
TSEncryptPassword(char
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/693
OK, I'll close this and think this issue again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project doe
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/693
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/693
Yes, this could increase time to first byte. We should send the first chunk
in anyway.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user masaori335 commented on the issue:
https://github.com/apache/trafficserver/pull/693
## before
```
$ nghttp --no-dep -vn
"https://127.0.0.1:4443/stream-bytes/163840?chunk_size=16284";
...
[ 0.130] recv DATA frame
[ 0.130] recv DATA frame
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/693
TS-4324: Buffer responses to not send small size DATA frames
If body_done flag is not marked and read available size is smaller
than 16KB in Http2ConnectionState::send_a_data_frame
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/686#discussion_r65574580
--- Diff: mgmt/api/INKMgmtAPI.cc ---
@@ -1865,8 +1865,7 @@ TSGetErrorMessage(TSMgmtError err_id)
tsapi TSMgmtError
TSEncryptPassword(char
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/686#discussion_r65416448
--- Diff: mgmt/WebMgmtUtils.h ---
@@ -113,7 +113,7 @@ bool recordRangeCheck(const char *pattern, const char
*value);
bool recordIPCheck(const
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/686
TS-4496: Remove ink_code_md5_stringify
[TS-4496](https://issues.apache.org/jira/browse/TS-4496)
You can merge this pull request into a Git repository by running:
$ git pull https
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/684
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/682
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/684
Yes! I'll file another JIRA tickets for it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project doe
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/684
TS-4494: Replace ink_code_md5_stringify to ink_code_to_hex_str
- [TS-4494](https://issues.apache.org/jira/browse/TS-4494)
- It looks like `ink_code_md5_stringify` is broken. So using
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/682
TS-4489: Remove second calling of Http2ConnectionState::delete_stream in
Http2Stream::do_io_close
[TS-4489](https://issues.apache.org/jira/browse/TS-4489)
You can merge this pull request
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/662
TS-4477: Decrement http_current_client_transactions_stat in HTTP/2
[TS-4477](https://issues.apache.org/jira/browse/TS-4477)
You can merge this pull request into a Git repository by
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/660
TS-4472: Decrement http_current_active_client_connections_stat in HTTP/2
`http_current_active_client_connections_stat` is incremented in HttpSM.
And if client connection is HTTP/1.1
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63741679
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-219591587
The new commit passed the build tests on the CI
- https://ci.trafficserver.apache.org/view/github/job/Github-Linux/80/
- https
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63255750
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -238,22 +241,29 @@ rcv_headers_frame(Http2ConnectionState &cstate, const
Http2Frame &a
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63215167
--- Diff: proxy/http2/Http2DependencyTree.h ---
@@ -0,0 +1,296 @@
+/** @file
+
+ HTTP/2 Dependency Tree
+
+ The original idea
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63212972
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1976,6 +1976,8 @@ static const RecordElement RecordsConfig[] =
//
{RECT_CONFIG
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63078637
--- Diff: proxy/http2/HTTP2.h ---
@@ -60,6 +60,10 @@ const uint32_t HTTP2_MAX_FRAME_SIZE = 16384;
const uint32_t HTTP2_HEADER_TABLE_SIZE = 4096
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-218822758
This is second patch after the #525 and TS-4295.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/632
TS-3535: Experimental Support of HTTP/2 Stream Priority feature
This is an experimental support of HTTP/2 Stream Priority feature.
- Add a option to enable this feature ( disabled
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/593#issuecomment-216432563
Here is the docs : [Internationalize Documentation - Update PO
files](https://cwiki.apache.org/confluence/display/TS/Internationalize+Documentation
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/600
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/600#issuecomment-214584746
Summary is added.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/600#issuecomment-214583503
When I send HTTP/2 request to ATS (built with debug option), I always see
this assertion is triggered.
It looks like `make check` and Regression Tests
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/600#issuecomment-214095730
@shinrich Could you take a look?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/600
TS-4355: Change assert condition for TS-3612
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/masaori335/trafficserver ts-4355
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/584#issuecomment-212708586
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/581
TS-4361: Remove TS_FETCH_EVENT handlers from Http2ClientSession
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/masaori335
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/525
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-211173098
Now, FetchSM and PluginVC is removed from HTTP/2 components by TS-3612. It
look like I need more works to rebase this on latest master. I'm going to
Github user masaori335 closed the pull request at:
https://github.com/apache/trafficserver/pull/540
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/537#issuecomment-206131655
@jpeach I didn't know we can use `TestBox.h` in Unit Tests:p I'll fix tests
to use it.
https://github.com/apache/traffics
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/537#issuecomment-206128440
@jpeach I agree with using many asserts is ugly. But if I understand
correctly, when we use `TestBox.h`, we need to write tests as Regression Tests.
Right
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/537#issuecomment-206096251
@maskit Comparator is added.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/537#discussion_r58005044
--- Diff: lib/ts/PriorityQueue.h ---
@@ -0,0 +1,207 @@
+/** @file
+
+ Priority Queue Implementation using Min Heap
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/537#issuecomment-203739388
@maskit Please take another look.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/537#discussion_r57996852
--- Diff: lib/ts/PriorityQueue.h ---
@@ -0,0 +1,191 @@
+/** @file
+
+ Priority Queue Implimentation using Min Heap
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/537#discussion_r57996306
--- Diff: lib/ts/PriorityQueue.h ---
@@ -0,0 +1,191 @@
+/** @file
+
+ Priority Queue Implimentation using Min Heap
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/537#discussion_r57996067
--- Diff: lib/ts/PriorityQueue.h ---
@@ -0,0 +1,191 @@
+/** @file
+
+ Priority Queue Implimentation using Min Heap
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/540
TS-4087: Docs: About new settings to limit streams
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/masaori335/trafficserver ts
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/539
TS-4298: Breakdown parameter checks in Http2::init()
[TS-4298](https://issues.apache.org/jira/browse/TS-4298)
You can merge this pull request into a Git repository by running:
$ git
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-200705387
I spined out priority queue stuff as
[TS-4295](https://issues.apache.org/jira/browse/TS-4295).
I'm going to fix this patch to use it and fix conf
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/537
TS-4295: Add Priority Queue in lib/ts/
[TS-4295](https://issues.apache.org/jira/browse/TS-4295)
You can merge this pull request into a Git repository by running:
$ git pull https
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-200337999
@PSUdaemon Yes, it is! It is one of reasons of implemented
Http2PriorityQueue using template.
Is it better to change name and move under `lib/ts
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-200325489
@zwoop Should we avoid STL in test code too? I used `ts/Vec.h` and
implemented priority queue to avoid STL:D
---
If your project is set up for it, you can
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/485#issuecomment-199581399
That is good point. This patch doesn't cover such cases.
> I think it would be better to dynamically adjust the max stream when a
new s
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/525
TS-3535: Experimental Support of Stream Priority Feature in HTTP/2
[TS-3535](https://issues.apache.org/jira/browse/TS-3535) Experimental
Support of Stream Priority Feature
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/522#issuecomment-194609719
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/485#issuecomment-188585450
1. Removed `max_active_streams_per_thread_in` and related code.
2. Set default value of `proxy.config.http2.max_active_streams_in` 0 for
backward
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/485#discussion_r54039631
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -1160,3 +1162,31 @@
Http2ConnectionState::send_window_update_frame(Http2StreamId id, uint32_t
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r53448759
--- Diff: proxy/http2/HTTP2.cc ---
@@ -484,246 +498,162 @@ convert_from_2_to_1_1_header(HTTPHdr *headers)
return PARSE_DONE
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r53417506
--- Diff: proxy/http2/HTTP2.cc ---
@@ -484,246 +498,162 @@ convert_from_2_to_1_1_header(HTTPHdr *headers)
return PARSE_DONE
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r53414542
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -992,43 +998,50 @@ Http2ConnectionState::send_headers_frame(FetchSM
*fetch_sm
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r53414029
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -992,43 +998,50 @@ Http2ConnectionState::send_headers_frame(FetchSM
*fetch_sm
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/485
TS-4087: Reduce SETTINGS_MAX_CONCURRENT_STREAMS when too many streams
Add below variables in records.config
- proxy.config.http2.min_concurrent_streams_in
- proxy.config.http2
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/484
TS-4213: Remove unused arguments from frame_handlers
[TS-4213](https://issues.apache.org/jira/browse/TS-4213)
You can merge this pull request into a Git repository by running:
$ git
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/480
TS-4209: Limit concurrent streams correctly
[TS-4209](https://issues.apache.org/jira/browse/TS-4209)
You can merge this pull request into a Git repository by running:
$ git pull
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/471#issuecomment-183182665
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/468#issuecomment-183179482
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-183178568
Please take another look
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/466#issuecomment-183178397
@bryancall OK :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/466#issuecomment-183175215
@bryancall Oh really? I'll check.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/466#discussion_r52704287
--- Diff: proxy/http2/HTTP2.cc ---
@@ -727,7 +727,7 @@ http2_decode_header_blocks(HTTPHdr *hdr, const uint8_t
*buf_start, const uint8_t
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-182204034
I didn't know this one. Thanks!
I'll fix with this.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-182203173
@jpeach I thought that too, but it has error in my box (CentOS7, gcc-4.8)
Diff: https://paste.apache.org/4tlm
Result: https://paste.apache.org/gnEb
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-182201082
(Because of I force pushed, James's comment is disappeared:\ )
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-182200775
@bryancall It looks good. But it totally needs 3 macros like below as far
as I tried.
```
#define DebugHttp2Con(ua_session, fmt
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-181738410
@jpeach Please take a look
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/454#issuecomment-180258234
I agree with we should keep consistency. So, I'm going to keep current
names and debug tag in Http2ClientSession.
The problem is same
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/454
TS-4172: Cleanup debug logs of Http2ConnectionState
[TS-4172](https://issues.apache.org/jira/browse/TS-4172)
You can merge this pull request into a Git repository by running:
$ git
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/435#issuecomment-178292993
Looks good to me.
@bgaff Could you take a look this? If you're ok, I'll land this.
---
If your project is set up for it, you can reply to this
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/444#issuecomment-177047293
Current value is 0, because active timeout of VConnection in
Http2ClientSession is not set.
I changed default value of `proxy.config.http2
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/444#discussion_r51245359
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1995,6 +1995,8 @@ static const RecordElement RecordsConfig[] =
,
{RECT_CONFIG
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/444
TS-4162: Add proxy.config.http2.active_timeout_in
- Fix [TS-4162](https://issues.apache.org/jira/browse/TS-4162)
You can merge this pull request into a Git repository by running
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/423
TS-4118: CID 1347871: Add error handling in HPACK_Encode
[TS-4118](https://issues.apache.org/jira/browse/TS-4118)
You can merge this pull request into a Git repository by running
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/406#issuecomment-170460942
I wrote about GitHub, ReadTheDocs and Transifex.
https://cwiki.apache.org/confluence/display/TS/Internationalize+Documentation
---
If your project is set
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/391#issuecomment-170210554
Oops, I mean TS-4092.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/391#issuecomment-170200715
I'm fine to ship this.
1. Actually I'm not big fun of names of classes and functions in HPACK
starting with 'HTTP2'. But those con
1 - 100 of 150 matches
Mail list logo
