[PR] Coverity 1517812: COPY_INSTEAD_OF_MOVE [trafficserver]

2024-05-16 Thread via GitHub
sharkleash opened a new pull request, #11361: URL: https://github.com/apache/trafficserver/pull/11361 This would empty the contents of block, do we want that? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] Coverity 1523638: Resource leak in object in cache test for RWW [trafficserver]

2024-05-16 Thread via GitHub
bryancall merged PR #11353: URL: https://github.com/apache/trafficserver/pull/11353 -- 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] 9.2.x: apply yapf to log-field-json.test.py [trafficserver]

2024-05-16 Thread via GitHub
bneradt merged PR #11360: URL: https://github.com/apache/trafficserver/pull/11360 -- 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] Remove the need of `-Wno-stringop-overflow` warning suppression [trafficserver]

2024-05-16 Thread via GitHub
maskit merged PR #11357: URL: https://github.com/apache/trafficserver/pull/11357 -- 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...@traf

Re: [PR] Remove the need of `-Wno-stringop-overflow` warning suppression [trafficserver]

2024-05-16 Thread via GitHub
freak82 commented on PR #11357: URL: https://github.com/apache/trafficserver/pull/11357#issuecomment-2114959455 I replaced the `strncat` usage in the `s3_auth` plugin with `TSstrlcat`. Tested the build again with both GCC 11, which was giving the warning in the plugin, and with GCC 13.

Re: [PR] Adding Cripts documentation structure [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11359: URL: https://github.com/apache/trafficserver/pull/11359#issuecomment-2114942722 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Remove obsolete debug output calls in 18 source files. [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11307: URL: https://github.com/apache/trafficserver/pull/11307#issuecomment-2114939933 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Eliminate leaking of HostStatusSync instances. [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11275: URL: https://github.com/apache/trafficserver/pull/11275#issuecomment-2114938988 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Remove obsolete debug output calls in 18 source files. [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11307: URL: https://github.com/apache/trafficserver/pull/11307#issuecomment-2114941327 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Fix undefined behavior doing memcpy from `timespec**` [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11356: URL: https://github.com/apache/trafficserver/pull/11356#issuecomment-2114925170 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Remove obsolete debug output calls in 23 source files. [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11298: URL: https://github.com/apache/trafficserver/pull/11298#issuecomment-2114928446 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Coverity 1534725: Uninitialized scalar variable [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11336: URL: https://github.com/apache/trafficserver/pull/11336#issuecomment-2114918070 Cherry-picked to v10.0.x -- 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 spec

Re: [PR] Use large IOBufferBlock only on sending DATA frame [trafficserver]

2024-05-16 Thread via GitHub
cmcfarlen commented on PR #11229: URL: https://github.com/apache/trafficserver/pull/11229#issuecomment-2114913932 Cherry-picked to v10.0.x -- 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 spec