Rescue Errno::EBADF in Client#close

[dentarg]

Spotted happening in CI: https://github.com/puma/puma/runs/4095295281?check_suite_focus=true

2021-11-03 16:34:54 +0000 Exception handling servers: #<Errno::EBADF: Bad file descriptor>/Users/runner/work/puma/puma/lib/puma/client.rb:163:in `close': Bad file descriptor (Errno::EBADF)
TestRackServer#test_large_post_body =
        from /Users/runner/work/puma/puma/lib/puma/client.rb:163:in `close'

Related to #2700? (that changed rescue Object to IOError, Errno::EBADF and re-raises these two)

See also #2312 that added Errno::EBADF (and #2745 that improves on that)

[MSP-Greg]

@dentarg

These macOS errors are getting a little frustrating. I had this and some other changes to test code, running more CI tonite. I don't think that I've seen the error on Ubuntu or Widows (or non-MRI?) , so I added a comment that Errno::EBADF was for darwin...

[dentarg]

@MSP-Greg Feel free to add/change anything in this PR if you want. I created this quickly to be able to discuss it.

Is some of the various exceptions we see in CI the fall out from #2700?

[MSP-Greg]

Is some of the various exceptions we see in CI the fall out from #2700?

I don't think so, but I'll look at it again. I'm not sure what the cause is, but most of the problems have been in integration tests, which involve sub-processes. I.think.