6.2.x branch

The 6.2.x branch has been updated with all issues that were marked for 6.2.x and resolved/closed. Please make sure your commits there look correct. Also, there are 5 issues still open that are marked for 6.2.0 backport and deemed blockers. We need to focus some energy here. https://issues.apache.o

[GitHub] trafficserver pull request: TS-4458: disabling configuration modif...

Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/651 --- 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 feature

[GitHub] trafficserver pull request: TS-4331Major re-write of hostdb

GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/653 TS-4331Major re-write of hostdb This is still a WIP, but I figure it's about time to get some feedback on the code here. The primary goal of this is to fix the crashing issues desc

[GitHub] trafficserver pull request: TS-4072 Diagnostic log rolling races

Github user danobi commented on the pull request: https://github.com/apache/trafficserver/pull/568#issuecomment-220702623 Fixed --- 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 en

[GitHub] trafficserver pull request: [TS-4408] Make volume.config not order...

Github user calavera commented on the pull request: https://github.com/apache/trafficserver/pull/618#issuecomment-220679375 @jpeach I made some changes in ConfigVolumes::BuildListFromString to handle the un-ordered options too. I could not find those generic parsing routines but I can

[GitHub] trafficserver pull request: TS-4459 - Force domain names in cert t...

Github user mjaniszewski commented on the pull request: https://github.com/apache/trafficserver/pull/647#issuecomment-220656079 The point regarding malloc/lowercase on each lookup is a good one, but indexing two separate versions of the cert won't cover all cases. Let's say a

[GitHub] trafficserver pull request: TS-4459 - Force domain names in cert t...

Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/647#issuecomment-220640925 I don't think that we should do the malloc and lowercasing on every lookup. How about this ... when the caller indexes the certificate, check whether the n

[GitHub] trafficserver pull request: TS-4448 Remove superfluous \n in Debug...

Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/652#issuecomment-220637257 +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 enabl

[GitHub] trafficserver pull request: [TS-4443] regex_remap: fix $i substitu...

Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/643 --- 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 feature i

[GitHub] trafficserver pull request: [TS-4443] regex_remap: fix $i substitu...

Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/643#issuecomment-220634880 looks good to me, passes CI build tests on Linux/FBSD. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel