Re: [PR] Add Sphinx documentation for statichit plugin. [trafficserver]

2024-05-23 Thread via GitHub
ywkaras commented on code in PR #11389: URL: https://github.com/apache/trafficserver/pull/11389#discussion_r1612485838 ## doc/admin-guide/plugins/statichit.en.rst: ## @@ -0,0 +1,106 @@ +.. _admin-plugins-statichit: +.. include:: ../../common.defs + +Static Hit Plugin +**

Re: [PR] Add Sphinx documentation for statichit plugin. [trafficserver]

2024-05-23 Thread via GitHub
ywkaras commented on PR #11389: URL: https://github.com/apache/trafficserver/pull/11389#issuecomment-2128193689 [approve ci docs] -- 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 comm

Re: [PR] Add http2.incomplete_header_timeout_in [trafficserver]

2024-05-23 Thread via GitHub
masaori335 merged PR #11354: URL: https://github.com/apache/trafficserver/pull/11354 -- 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: github-unsubscr...@

Re: [PR] Add Sphinx documentation for statichit plugin. [trafficserver]

2024-05-23 Thread via GitHub
mlibbey commented on code in PR #11389: URL: https://github.com/apache/trafficserver/pull/11389#discussion_r1612438247 ## doc/admin-guide/plugins/statichit.en.rst: ## @@ -0,0 +1,106 @@ +.. _admin-plugins-statichit: +.. include:: ../../common.defs + +Static Hit Plugin +**

Re: [PR] Downgrade luajit to 2.1_p20230410-r3 [trafficserver-ingress-controller]

2024-05-23 Thread via GitHub
shukitchan merged PR #230: URL: https://github.com/apache/trafficserver-ingress-controller/pull/230 -- 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: gith

[PR] Temporary solution to hex/unhex for Cripts [trafficserver]

2024-05-23 Thread via GitHub
zwoop opened a new pull request, #11390: URL: https://github.com/apache/trafficserver/pull/11390 (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 unsubscribe

Re: [PR] Make sure to use proper allocator to free [trafficserver]

2024-05-23 Thread via GitHub
bneradt commented on PR #11381: URL: https://github.com/apache/trafficserver/pull/11381#issuecomment-2127891226 [approve ci autest] -- 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 co

Re: [PR] Upgrade to alpine linux 3.20.0 [trafficserver-ingress-controller]

2024-05-23 Thread via GitHub
shukitchan merged PR #229: URL: https://github.com/apache/trafficserver-ingress-controller/pull/229 -- 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: gith

Re: [PR] Coverity 1545946: Uninitialized scalar variable in server transform plugin example [trafficserver]

2024-05-23 Thread via GitHub
bryancall merged PR #11387: URL: https://github.com/apache/trafficserver/pull/11387 -- 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: github-unsubscr...@t

Re: [PR] Coverity 1523684: Resource leak [trafficserver]

2024-05-23 Thread via GitHub
serrislew commented on PR #11338: URL: https://github.com/apache/trafficserver/pull/11338#issuecomment-2127709306 [approve ci autest] -- 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

[PR] Upgrade to alpine linux 3.20.0 [trafficserver-ingress-controller]

2024-05-23 Thread via GitHub
shukitchan opened a new pull request, #229: URL: https://github.com/apache/trafficserver-ingress-controller/pull/229 (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 comm

Re: [PR] Pass url_len when emplacing in Http2CommonSession::_h2_pushed_urls [trafficserver]

2024-05-23 Thread via GitHub
ywkaras merged PR #11382: URL: https://github.com/apache/trafficserver/pull/11382 -- 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: github-unsubscr...@tra

Re: [PR] Pass url_len when emplacing in Http2CommonSession::_h2_pushed_urls [trafficserver]

2024-05-23 Thread via GitHub
ywkaras commented on PR #11382: URL: https://github.com/apache/trafficserver/pull/11382#issuecomment-2127031313 Yes, it' s confusion, it's not done consistently. -- 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: [PR] Unit test: enable plugin dso logic to build and run unit tests. [trafficserver]

2024-05-23 Thread via GitHub
brbzull0 commented on PR #11367: URL: https://github.com/apache/trafficserver/pull/11367#issuecomment-2126986170 [approve ci osx] -- 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 comm

Re: [PR] Unit test: enable plugin dso logic to build and run unit tests. [trafficserver]

2024-05-23 Thread via GitHub
brbzull0 commented on PR #11367: URL: https://github.com/apache/trafficserver/pull/11367#issuecomment-2126986424 [approve ci autest] -- 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 c