summaryrefslogtreecommitdiff
path: root/cmd/caddy
diff options
context:
space:
mode:
authorMatthew Holt <mholt@users.noreply.github.com>2022-01-10 23:24:58 -0700
committerMatthew Holt <mholt@users.noreply.github.com>2022-01-10 23:24:58 -0700
commit64a3218f5c7ba98a51c76065ed32ae860351e779 (patch)
tree7c73bf805d6f0cf483e86e6bcba94bd8527b44e5 /cmd/caddy
parentc634bbe9cc7ef6ce6f9f776010ce96384fd43340 (diff)
core: Simplify shared listeners, fix deadline bug
When this listener code was first written, UsagePool didn't exist. We can simplify much of the wrapped listener logic by utilizing UsagePool. This also fixes a bug where new servers were able to clear deadlines set by old servers, even if the old server didn't get booted out of its Accept() call yet. And with the deadline cleared, they never would. (Sometimes. Based on reports and difficulty of reproducing the bug, this behavior was extremely rare.) I don't know why that happened exactly, maybe some polling mechanism in the kernel and if the timings worked out just wrong it would expose the bug. Anyway, now we ensure that only the closer that set the deadline is the same one that clears it, ensuring that old servers always return out of Accept(), because the deadline doesn't get cleared until they do. Of course, all this hinges on the hope that my suspicions in the middle of the night are correct and that kernels work the way I think they do in my head. Also minor enhancement to UsagePool where if a value errors upon construction (a very real possibility with listeners), it is removed from the pool. Not 100% sure the sync logic is correct there, or maybe we don't have to even put it in the pool until after construction, but it's subtle either way and I think this is safe... right?
Diffstat (limited to 'cmd/caddy')
0 files changed, 0 insertions, 0 deletions