[I] cachekey plugin throws error if remap based cachekey is configured [trafficserver]

2024-07-08 Thread via GitHub
mat1010 opened a new issue, #11516: URL: https://github.com/apache/trafficserver/issues/11516 We are in a situation where we have some remap rules configure that include cookies for specific domains. Since we do not want to place a cachekey configuration to each remap rule, we also

Re: [I] Using remap to whitelist hostnames is not working with HTTPS on some hosts [trafficserver]

2024-07-06 Thread via GitHub
github-actions[bot] commented on issue #9949: URL: https://github.com/apache/trafficserver/issues/9949#issuecomment-2212246537 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] ATS will not build with VERIFY_JTEST_DATA defined [trafficserver]

2024-07-05 Thread via GitHub
JosiahWI closed issue #11455: ATS will not build with VERIFY_JTEST_DATA defined URL: https://github.com/apache/trafficserver/issues/11455 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] assert on POST when the data is bigger than the used buffer. [trafficserver]

2024-07-05 Thread via GitHub
maskit closed issue #9797: assert on POST when the data is bigger than the used buffer. URL: https://github.com/apache/trafficserver/issues/9797 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] How to config ATS to refresh cache for files modified on server [trafficserver]

2024-07-04 Thread via GitHub
mlibbey closed issue #9939: How to config ATS to refresh cache for files modified on server URL: https://github.com/apache/trafficserver/issues/9939 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [I] How to config ATS to refresh cache for files modified on server [trafficserver]

2024-07-04 Thread via GitHub
mlibbey commented on issue #9939: URL: https://github.com/apache/trafficserver/issues/9939#issuecomment-2209451007 Think we can close this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] How to config ATS to refresh cache for files modified on server [trafficserver]

2024-07-03 Thread via GitHub
github-actions[bot] commented on issue #9939: URL: https://github.com/apache/trafficserver/issues/9939#issuecomment-2207857711 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] Some debug stats are counted under the wrong metrics [trafficserver]

2024-07-03 Thread via GitHub
JosiahWI closed issue #11457: Some debug stats are counted under the wrong metrics URL: https://github.com/apache/trafficserver/issues/11457 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] Some debug stats are counted under the wrong metrics [trafficserver]

2024-07-03 Thread via GitHub
JosiahWI closed issue #11457: Some debug stats are counted under the wrong metrics URL: https://github.com/apache/trafficserver/issues/11457 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] HTTP/3 benchmark results [trafficserver]

2024-07-03 Thread via GitHub
brbzull0 closed issue #11446: HTTP/3 benchmark results URL: https://github.com/apache/trafficserver/issues/11446 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] Assertion failure in HttpSM::tunnel_handler_post_ua [trafficserver]

2024-07-02 Thread via GitHub
maskit commented on issue #9229: URL: https://github.com/apache/trafficserver/issues/9229#issuecomment-2204381446 I can't reproduce the assertion failure on the latest master. It might have been fixed by some changes. -- This is an automated message from the Apache Git Service. To

Re: [I] Assertion failure in HttpSM::tunnel_handler_post_ua [trafficserver]

2024-07-02 Thread via GitHub
maskit closed issue #9229: Assertion failure in HttpSM::tunnel_handler_post_ua URL: https://github.com/apache/trafficserver/issues/9229 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] forward_route: use close_notify on shutdown [trafficserver]

2024-06-21 Thread via GitHub
github-actions[bot] commented on issue #9880: URL: https://github.com/apache/trafficserver/issues/9880#issuecomment-2177370738 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] cache write lock not released when following redirect and no parent [trafficserver]

2024-06-21 Thread via GitHub
github-actions[bot] commented on issue #9275: URL: https://github.com/apache/trafficserver/issues/9275#issuecomment-2177371132 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

[I] Some debug stats are counted under the wrong metrics [trafficserver]

2024-06-18 Thread via GitHub
JosiahWI opened a new issue, #11457: URL: https://github.com/apache/trafficserver/issues/11457 The one I noticed is in agg_copy; in debug mode, every time bytes are copied to the aggregation buffer to prepare for writing, it's counted as a backlog failure instead of a write. The cache will

Re: [I] Status mismatch in header_rewrite [trafficserver]

2024-06-17 Thread via GitHub
jasmine-nahrain commented on issue #11397: URL: https://github.com/apache/trafficserver/issues/11397#issuecomment-2174533415 Can confirm behaviour is the same in 8.1.x -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] Missing git tag for 9.2.4 [trafficserver]

2024-06-17 Thread via GitHub
bryancall closed issue #11436: Missing git tag for 9.2.4 URL: https://github.com/apache/trafficserver/issues/11436 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] Missing git tag for 9.2.4 [trafficserver]

2024-06-17 Thread via GitHub
bryancall commented on issue #11436: URL: https://github.com/apache/trafficserver/issues/11436#issuecomment-2174498822 @ezelkow1 created the tag -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[I] ATS will not build with VERIFY_JTEST_DATA defined [trafficserver]

2024-06-17 Thread via GitHub
JosiahWI opened a new issue, #11455: URL: https://github.com/apache/trafficserver/issues/11455 ``` b5faf95ecc11:/trafficserver # cmake --build build --target inkcache [0/2] Re-checking globbed directories... [302/306] Building CXX object

Re: [I] Records parses uninitialized data for IP addresses [trafficserver]

2024-06-17 Thread via GitHub
JosiahWI closed issue #11449: Records parses uninitialized data for IP addresses URL: https://github.com/apache/trafficserver/issues/11449 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] Records parses uninitialized data for IP addresses [trafficserver]

2024-06-17 Thread via GitHub
JosiahWI closed issue #11449: Records parses uninitialized data for IP addresses URL: https://github.com/apache/trafficserver/issues/11449 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] HTTP/3 benchmark results [trafficserver]

2024-06-16 Thread via GitHub
brbzull0 commented on issue #11446: URL: https://github.com/apache/trafficserver/issues/11446#issuecomment-2171781091 it seems aren't deleting any streams at all. probably because we do not get the right answer from this:

Re: [I] H2 POST request causes crash [trafficserver]

2024-06-15 Thread via GitHub
github-actions[bot] commented on issue #9857: URL: https://github.com/apache/trafficserver/issues/9857#issuecomment-2171009247 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

[I] Records parses uninitialized data for IP addresses [trafficserver]

2024-06-14 Thread via GitHub
JosiahWI opened a new issue, #11449: URL: https://github.com/apache/trafficserver/issues/11449 This issue was detected by valgrind. This is the relevant part of the code, with comments added to show where things go wrong. ```cpp void RecHttpLoadIpAddrsFromConfVar(const char

Re: [I] HTTP/3 benchmark results [trafficserver]

2024-06-13 Thread via GitHub
brbzull0 commented on issue #11446: URL: https://github.com/apache/trafficserver/issues/11446#issuecomment-2165982326 I was just talking with Masakazu, I'll start looking into this now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] HTTP/3 benchmark results [trafficserver]

2024-06-13 Thread via GitHub
bryancall commented on issue #11446: URL: https://github.com/apache/trafficserver/issues/11446#issuecomment-2165970745 Changes for benchmarking and debugging the issue: ``` diff --git a/src/iocore/net/QUICPacketHandler.cc b/src/iocore/net/QUICPacketHandler.cc index

Re: [I] HTTP/3 benchmark results [trafficserver]

2024-06-13 Thread via GitHub
bryancall commented on issue #11446: URL: https://github.com/apache/trafficserver/issues/11446#issuecomment-2165958277 These are the results of http/2: ``` **http2load** finished in 65.00s, 781381.18 req/s, 783.06MB/s requests: 46882871 total, 46882871 started, 46882871

[I] HTTP/3 benchmark results [trafficserver]

2024-06-13 Thread via GitHub
bryancall opened a new issue, #11446: URL: https://github.com/apache/trafficserver/issues/11446 ``` **http2load** finished in 65.00s, 25725.23 req/s, 29.04MB/s requests: 1543514 total, 1543514 started, 1543514 done, 1543514 succeeded, 0 failed, 0 errored, 0 timeout status codes:

Re: [I] Potentially problematic functions ` LogAccess::unmarshal_int_to_netscape|date|time_str` [trafficserver]

2024-06-12 Thread via GitHub
ywkaras closed issue #11364: Potentially problematic functions ` LogAccess::unmarshal_int_to_netscape|date|time_str` URL: https://github.com/apache/trafficserver/issues/11364 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [I] Binary Equivalence Problem [trafficserver]

2024-06-12 Thread via GitHub
github-actions[bot] commented on issue #8175: URL: https://github.com/apache/trafficserver/issues/8175#issuecomment-2163861062 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] asking for NVDIMM cache layer enablement [trafficserver]

2024-06-12 Thread via GitHub
github-actions[bot] commented on issue #8693: URL: https://github.com/apache/trafficserver/issues/8693#issuecomment-2163860968 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] Update build h3 script to have the option to build quiche with openssl/quictls [trafficserver]

2024-06-12 Thread via GitHub
brbzull0 closed issue #10873: Update build h3 script to have the option to build quiche with openssl/quictls URL: https://github.com/apache/trafficserver/issues/10873 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] Update build h3 script to have the option to build quiche with openssl/quictls [trafficserver]

2024-06-12 Thread via GitHub
brbzull0 commented on issue #10873: URL: https://github.com/apache/trafficserver/issues/10873#issuecomment-2162337806 closed by https://github.com/apache/trafficserver/pull/11270 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] Is there any way to disable back to the source function? [trafficserver]

2024-06-11 Thread via GitHub
github-actions[bot] commented on issue #9796: URL: https://github.com/apache/trafficserver/issues/9796#issuecomment-2161934747 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] autest command errors should log the failing command [trafficserver]

2024-06-11 Thread via GitHub
github-actions[bot] commented on issue #9772: URL: https://github.com/apache/trafficserver/issues/9772#issuecomment-2161934842 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] Intermittent CI failures of ja3_fingerprint [trafficserver]

2024-06-11 Thread via GitHub
JosiahWI closed issue #11431: Intermittent CI failures of ja3_fingerprint URL: https://github.com/apache/trafficserver/issues/11431 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] Is this incorrect initialization of `LogField::m_set_func`? [trafficserver]

2024-06-11 Thread via GitHub
ywkaras closed issue #11373: Is this incorrect initialization of `LogField::m_set_func`? URL: https://github.com/apache/trafficserver/issues/11373 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [I] Mutex use in APIHook::invoke() seems moot and crash prone. [trafficserver]

2024-06-10 Thread via GitHub
ywkaras closed issue #8903: Mutex use in APIHook::invoke() seems moot and crash prone. URL: https://github.com/apache/trafficserver/issues/8903 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] Why are we linking libtsutil.a into all the core plugins? [trafficserver]

2024-06-10 Thread via GitHub
ywkaras commented on issue #11032: URL: https://github.com/apache/trafficserver/issues/11032#issuecomment-2159538859 I guess this is fixed/moot now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [I] Why are we linking libtsutil.a into all the core plugins? [trafficserver]

2024-06-10 Thread via GitHub
ywkaras closed issue #11032: Why are we linking libtsutil.a into all the core plugins? URL: https://github.com/apache/trafficserver/issues/11032 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] What is the current status of http3 support in ATS 10.x? [trafficserver]

2024-06-10 Thread via GitHub
bryancall closed issue #11422: What is the current status of http3 support in ATS 10.x? URL: https://github.com/apache/trafficserver/issues/11422 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] Missed to pass `length` parameter in the tscpp api - `HeaderField::append` [trafficserver]

2024-06-10 Thread via GitHub
bryancall closed issue #11395: Missed to pass `length` parameter in the tscpp api - `HeaderField::append` URL: https://github.com/apache/trafficserver/issues/11395 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [I] Build without hwloc broken on master [trafficserver]

2024-06-10 Thread via GitHub
JosiahWI closed issue #11434: Build without hwloc broken on master URL: https://github.com/apache/trafficserver/issues/11434 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[I] Missing git tag for 9.2.4 [trafficserver]

2024-06-10 Thread via GitHub
mraerino opened a new issue, #11436: URL: https://github.com/apache/trafficserver/issues/11436 According to the [download page](https://trafficserver.apache.org/downloads), the latest release is `9.2.4` but it seems this version has not been tagged in git. I can only see the `9.2.4-rc0`

Re: [I] 10.0.x: OCSP errors on startup [trafficserver]

2024-06-08 Thread via GitHub
github-actions[bot] commented on issue #9819: URL: https://github.com/apache/trafficserver/issues/9819#issuecomment-2156268255 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

[I] Intermittent CI failures of ja3_fingerprint [trafficserver]

2024-06-08 Thread via GitHub
JosiahWI opened a new issue, #11431: URL: https://github.com/apache/trafficserver/issues/11431 After investigating the CI failures, I discovered that they are due to a race condition between the `DUMP_HEADER` macro and some other debug logging in `HttpTransact`. The debug output can be

Re: [I] certifier plugin crash on master [trafficserver]

2024-06-07 Thread via GitHub
github-actions[bot] commented on issue #9794: URL: https://github.com/apache/trafficserver/issues/9794#issuecomment-2155752411 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] traffic_layout remove command core dumped [trafficserver]

2024-06-06 Thread via GitHub
maskit commented on issue #9065: URL: https://github.com/apache/trafficserver/issues/9065#issuecomment-2153472988 I was able to reproduce the crash on master branch. It looks like `traffic_layout` assumes these directories below are under a run-root directory. If some of them are not

Re: [I] Cache unit tests failed on debian ci build [trafficserver]

2024-06-06 Thread via GitHub
JosiahWI commented on issue #11372: URL: https://github.com/apache/trafficserver/issues/11372#issuecomment-2153373194 The first of the two failures may have been fixed by #11347. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] What is the current status of http3 support in ATS 10.x? [trafficserver]

2024-06-06 Thread via GitHub
maskit commented on issue #11422: URL: https://github.com/apache/trafficserver/issues/11422#issuecomment-2153055251 It's still experimental support on either branch. There is no difference (or only tiny difference) between 10.0.x and master in terms of HTTP/3 at the moment. If you are

Re: [I] What branch should I use to try ATS 10.x features? [trafficserver]

2024-06-06 Thread via GitHub
ksqrtr commented on issue #11363: URL: https://github.com/apache/trafficserver/issues/11363#issuecomment-2152955929 thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [I] What branch should I use to try ATS 10.x features? [trafficserver]

2024-06-06 Thread via GitHub
ksqrtr closed issue #11363: What branch should I use to try ATS 10.x features? URL: https://github.com/apache/trafficserver/issues/11363 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? [trafficserver]

2024-06-05 Thread via GitHub
ywkaras closed issue #11374: Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? URL: https://github.com/apache/trafficserver/issues/11374 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [I] trafficserver 9.1.1 traffic_server ERROR: plugin 'regex_remap.so' failed to load: [trafficserver]

2024-06-04 Thread via GitHub
github-actions[bot] commented on issue #9750: URL: https://github.com/apache/trafficserver/issues/9750#issuecomment-2148711361 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] 9.2.0: tons of HTTPInfo::response unmarshal failed and Invalid object length for deserialization0 [trafficserver]

2024-06-04 Thread via GitHub
github-actions[bot] commented on issue #9752: URL: https://github.com/apache/trafficserver/issues/9752#issuecomment-2148711310 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] Autest tls_tunnel fails because of a slightly different error message from python [trafficserver]

2024-06-04 Thread via GitHub
maskit closed issue #11409: Autest tls_tunnel fails because of a slightly different error message from python URL: https://github.com/apache/trafficserver/issues/11409 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] Test run in Au test tls_tunnel causing exception in proxy_protocol_client.py [trafficserver]

2024-06-04 Thread via GitHub
maskit commented on issue #9550: URL: https://github.com/apache/trafficserver/issues/9550#issuecomment-2147871340 @masaori335 I can't tell if it's related without seeing the actual error log, but this issue is going to be closed by #11415. Create a new issue if you still see the error you

Re: [I] Autest tls_tunnel fails because of a slightly different error message from python [trafficserver]

2024-06-04 Thread via GitHub
maskit commented on issue #11409: URL: https://github.com/apache/trafficserver/issues/11409#issuecomment-2147863526 Yes, looks like the same as the issue you reported. The one Masaori mentioned seems like a different issue. -- This is an automated message from the Apache Git Service. To

Re: [PR] git: Ignore build* [trafficserver-libswoc]

2024-06-03 Thread via GitHub
masaori335 closed pull request #4: git: Ignore build* URL: https://github.com/apache/trafficserver-libswoc/pull/4 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] Autest tls_tunnel fails because of a slightly different error message from python [trafficserver]

2024-06-03 Thread via GitHub
ywkaras commented on issue #11409: URL: https://github.com/apache/trafficserver/issues/11409#issuecomment-2146456911 Is this related to https://github.com/apache/trafficserver/issues/9550 ? -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] Status mismatch in header_rewrite [trafficserver]

2024-06-03 Thread via GitHub
bryancall commented on issue #11397: URL: https://github.com/apache/trafficserver/issues/11397#issuecomment-2146255051 Verify the behavior for 8.x and if it is the same as 9.x, then we should document it. -- This is an automated message from the Apache Git Service. To respond to the

Re: [I] Autest for static hit plugin fails on release build [trafficserver]

2024-06-03 Thread via GitHub
ywkaras closed issue #11403: Autest for static hit plugin fails on release build URL: https://github.com/apache/trafficserver/issues/11403 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] Autest tls_tunnel fails because of a slightly different error message from python [trafficserver]

2024-06-03 Thread via GitHub
bryancall commented on issue #11409: URL: https://github.com/apache/trafficserver/issues/11409#issuecomment-2146249025 @maskit Ask for @bneradt's help -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [I] ATS 7.1.2 crash frequently on CentOS 7.4 64bit [trafficserver]

2024-06-02 Thread via GitHub
kougazhang commented on issue #3169: URL: https://github.com/apache/trafficserver/issues/3169#issuecomment-2144189683 Maybe you can check kernel parameters related to virtual memory management in your Linux system. Increasing `vm.max_map_count` soloved my problem. -- This is an

Re: [I] AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write [trafficserver]

2024-05-30 Thread via GitHub
bneradt closed issue #11388: AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write URL: https://github.com/apache/trafficserver/issues/11388 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write [trafficserver]

2024-05-30 Thread via GitHub
bneradt closed issue #11388: AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write URL: https://github.com/apache/trafficserver/issues/11388 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Add CTest support [trafficserver-libswoc]

2024-05-30 Thread via GitHub
bneradt merged PR #3: URL: https://github.com/apache/trafficserver-libswoc/pull/3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [I] Missed to pass `length` parameter in the tscpp api - `HeaderField::append` [trafficserver]

2024-05-30 Thread via GitHub
freak82 commented on issue #11395: URL: https://github.com/apache/trafficserver/issues/11395#issuecomment-2138951083 Just a bit of clarification about these issues for missed to pss parameters. I've patches which allow the `-Wno-unused-parameter` warning suppression to be removed. I

Re: [I] Is there interest to get rid of `-Wno-unused-parameter` warning suppression? [trafficserver]

2024-05-30 Thread via GitHub
brbzull0 closed issue #11377: Is there interest to get rid of `-Wno-unused-parameter` warning suppression? URL: https://github.com/apache/trafficserver/issues/11377 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] git: Ignore build* [trafficserver-libswoc]

2024-05-30 Thread via GitHub
masaori335 opened a new pull request, #4: URL: https://github.com/apache/trafficserver-libswoc/pull/4 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Add CTest support [trafficserver-libswoc]

2024-05-30 Thread via GitHub
masaori335 opened a new pull request, #3: URL: https://github.com/apache/trafficserver-libswoc/pull/3 ``` ctest --test-dir ./build -T test ``` will execute `test_libswoc`. `BUILD_TESTING` is the replacement of `LIBSWOC_TEST`. -- This is an automated message from the Apache

[I] Status mismatch in header_rewrite [trafficserver]

2024-05-28 Thread via GitHub
jasmine-nahrain opened a new issue, #11397: URL: https://github.com/apache/trafficserver/issues/11397 The status parsed through resources in the exec func of a header_rewrite config can be different to the status seen in headers. e.g. ``` Config #1 cond

[I] Missed to pass `length` parameter in the tscpp api - `HeaderField::append` [trafficserver]

2024-05-25 Thread via GitHub
freak82 opened a new issue, #11395: URL: https://github.com/apache/trafficserver/issues/11395 Hi there, The [HeaderField::append](https://github.com/apache/trafficserver/blob/b46f5d061af455dc75956346da3308788381b30d/src/tscpp/api/Headers.cc#L310) function overload doesn't pass the

Re: [I] QPACK: Update Dynamic Table logics to the latest spec [trafficserver]

2024-05-24 Thread via GitHub
github-actions[bot] commented on issue #9747: URL: https://github.com/apache/trafficserver/issues/9747#issuecomment-2130656275 This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

Re: [I] Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? [trafficserver]

2024-05-24 Thread via GitHub
maskit commented on issue #11374: URL: https://github.com/apache/trafficserver/issues/11374#issuecomment-2129911911 Ah, I missed the use in `Handler::data`. Then just from the `TSIOBuffer` version. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [I] Clean up github actions [trafficserver-ingress-controller]

2024-05-24 Thread via GitHub
shukitchan closed issue #214: Clean up github actions URL: https://github.com/apache/trafficserver-ingress-controller/issues/214 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [I] Clean up github actions [trafficserver-ingress-controller]

2024-05-24 Thread via GitHub
shukitchan commented on issue #214: URL: https://github.com/apache/trafficserver-ingress-controller/issues/214#issuecomment-2129840710 All complete except lua related jobs -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [I] Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? [trafficserver]

2024-05-23 Thread via GitHub
freak82 commented on issue #11374: URL: https://github.com/apache/trafficserver/issues/11374#issuecomment-2128486018 It seems to me that this version of the function `uint64_t read(const TSIOBufferReader , std::string , int64_t l = 0)` is called by `void Handler::data(const

[I] Add AuTest to exercise http2.incomplete_header_timeout_in [trafficserver]

2024-05-23 Thread via GitHub
masaori335 opened a new issue, #11391: URL: https://github.com/apache/trafficserver/issues/11391 It looks like we need to find or write a client to exercise the timeout. https://github.com/apache/trafficserver/pull/11354 -- This is an automated message from the Apache Git Service. To

Re: [I] AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write [trafficserver]

2024-05-23 Thread via GitHub
bneradt commented on issue #11388: URL: https://github.com/apache/trafficserver/issues/11388#issuecomment-2128097652 Our ATS deployment is stable if I revert this PR locally: #10997. The patch is mostly code shuffling so should be safe. I'll work on trying to find out what in there is

Re: [I] Is this missed to forward argument in `Http2CommonSession::add_url_to_pushed_table`? [trafficserver]

2024-05-23 Thread via GitHub
ywkaras closed issue #11375: Is this missed to forward argument in `Http2CommonSession::add_url_to_pushed_table`? URL: https://github.com/apache/trafficserver/issues/11375 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[I] AddressSanitizer: heap-buffer-overflow on address in XpackDynamicTableStorage::write [trafficserver]

2024-05-22 Thread via GitHub
bneradt opened a new issue, #11388: URL: https://github.com/apache/trafficserver/issues/11388 Running https://github.com/apache/trafficserver/commit/399d04ceace14fae66dc615d96ff93557b55ad00 in production under ASan, we see the following ASan buffer overflow error:

Re: [I] Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? [trafficserver]

2024-05-22 Thread via GitHub
maskit commented on issue #11374: URL: https://github.com/apache/trafficserver/issues/11374#issuecomment-2125207293 It looks like the both `read` functions are only used to read whole data in a buffer. Forwarding the `l` argument is a valid change, but I'd remove the `l` arguments from

Re: [I] In what cases are updates by TSSslSecretSet() actually used? [trafficserver]

2024-05-21 Thread via GitHub
thebadpete commented on issue #9562: URL: https://github.com/apache/trafficserver/issues/9562#issuecomment-2123163629 Hi, this is Peter from Yahoo, we are seeing the issues in our prod hosts. It used to work some time back last year, but now we are seeing the issues everything there are

Re: [I] Is there interest to get rid of `-Wno-unused-parameter` warning suppression? [trafficserver]

2024-05-21 Thread via GitHub
maskit commented on issue #11377: URL: https://github.com/apache/trafficserver/issues/11377#issuecomment-2123020662 > I just believe that warning suppression is bad because the warnings sometimes point to real issues/bugs and we should use any help from the compiler and the other tools to

Re: [I] Is this missed to forward argument in `Http2CommonSession::add_url_to_pushed_table`? [trafficserver]

2024-05-21 Thread via GitHub
maskit commented on issue #11375: URL: https://github.com/apache/trafficserver/issues/11375#issuecomment-2122905860 Yes, it looks like a bug. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[I] Logs don't display URL properly for plugins [trafficserver]

2024-05-21 Thread via GitHub
smalenfant opened a new issue, #11378: URL: https://github.com/apache/trafficserver/issues/11378 Migrating from 8.1.x to 9.2.4, I see that we are losing the url hostname used to reach out for plugins. Maybe there is a missing configuration, but this used to work in the past. ```

[I] Is there interest to get rid of `-Wno-unused-parameter` warning suppression? [trafficserver]

2024-05-21 Thread via GitHub
freak82 opened a new issue, #11377: URL: https://github.com/apache/trafficserver/issues/11377 Hi there, So, I did the above experiment for myself and there are lots of places which need to be fixed. However, the warnings also revealed few places with potential issues/bugs and

[I] Possibly forgotten parameter of `Matcher::test_reg` in `plugins/header_rewrite /matcher.h` [trafficserver]

2024-05-21 Thread via GitHub
freak82 opened a new issue, #11376: URL: https://github.com/apache/trafficserver/issues/11376 Hi there, There is [this function](https://github.com/apache/trafficserver/blob/d31a40fecb19f80ae8b7ddbb2ff0efecc57bf7f9/plugins/header_rewrite/matcher.h#L189) and it has the following

[I] Is this missed to forward argument in `Http2CommonSession::add_url_to_pushed_table`? [trafficserver]

2024-05-21 Thread via GitHub
freak82 opened a new issue, #11375: URL: https://github.com/apache/trafficserver/issues/11375 Hi there, The `url_len` argument of `Http2CommonSession::add_url_to_pushed_table`

[I] Is this missed to pass argument in `plugins/multiplexer/dispatch.cc`? [trafficserver]

2024-05-21 Thread via GitHub
freak82 opened a new issue, #11374: URL: https://github.com/apache/trafficserver/issues/11374 Hi there, There is [this `read` function](https://github.com/apache/trafficserver/blob/master/plugins/multiplexer/dispatch.cc#L133) in `plugins/multiplexer/dispatch.cc`: ``` uint64_t

[I] Is this incorrect initialization of `LogField::m_set_func`? [trafficserver]

2024-05-21 Thread via GitHub
freak82 opened a new issue, #11373: URL: https://github.com/apache/trafficserver/issues/11373 Hi there, [This constructor of LogField](https://github.com/apache/trafficserver/blob/master/src/proxy/logging/LogField.cc#L330) takes `SetFunc _setfunc` but it doesn't set its member

[I] Cache unit tests failed on debian ci build [trafficserver]

2024-05-21 Thread via GitHub
masaori335 opened a new issue, #11372: URL: https://github.com/apache/trafficserver/issues/11372 I saw at least two failure on debian job of Pull-Request Builder. Did we change something? ``` 99% tests passed, 1 tests failed out of 138 Total Test time (real) = 30.24 sec

Re: [I] Question about the new `Metrics` stats interface [trafficserver]

2024-05-20 Thread via GitHub
freak82 closed issue #11349: Question about the new `Metrics` stats interface URL: https://github.com/apache/trafficserver/issues/11349 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] Question about the new `Metrics` stats interface [trafficserver]

2024-05-20 Thread via GitHub
freak82 commented on issue #11349: URL: https://github.com/apache/trafficserver/issues/11349#issuecomment-2121782128 Thank you all for the clarifications. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [I] Question about the new `Metrics` stats interface [trafficserver]

2024-05-20 Thread via GitHub
zwoop commented on issue #11349: URL: https://github.com/apache/trafficserver/issues/11349#issuecomment-2121360353 The common use case, using these APIs rather than the old APIs in ts.h (which also uses the new Metrics under the hood) would be ``` auto id1 =

Re: [I] The Apache Traffic Server is not listening on port 80 after starting the service ( service is active! ) [trafficserver]

2024-05-20 Thread via GitHub
bryancall closed issue #11292: The Apache Traffic Server is not listening on port 80 after starting the service ( service is active! ) URL: https://github.com/apache/trafficserver/issues/11292 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [I] Question about the new `Metrics` stats interface [trafficserver]

2024-05-20 Thread via GitHub
bryancall commented on issue #11349: URL: https://github.com/apache/trafficserver/issues/11349#issuecomment-2121343362 We require rebuilding plugin for new major version releases of ATS. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] Event handles with VC_EVENT_ERROR but not VC_EVENT_EOS [trafficserver]

2024-05-20 Thread via GitHub
bryancall closed issue #11358: Event handles with VC_EVENT_ERROR but not VC_EVENT_EOS URL: https://github.com/apache/trafficserver/issues/11358 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [I] What branch should I use to try ATS 10.x features? [trafficserver]

2024-05-20 Thread via GitHub
bryancall commented on issue #11363: URL: https://github.com/apache/trafficserver/issues/11363#issuecomment-2121333828 Please use the `10.0.x` branch -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

  1   2   3   4   5   6   7   8   9   10   >