Loggers - internal_write - catch Errno::EINVAL

[MSP-Greg]

Description

The following appeared in a recent CI job:

D:/a/puma/puma/lib/puma/log_writer.rb:75:in `flush': Invalid argument (Errno::EINVAL)
  from D:/a/puma/puma/lib/puma/log_writer.rb:75:in `internal_write'
  from D:/a/puma/puma/lib/puma/log_writer.rb:62:in `log'
  from D:/a/puma/puma/lib/puma/launcher.rb:321:in `log'
  from D:/a/puma/puma/lib/puma/launcher.rb:414:in `rescue in setup_signals'

The error raised can happen in the internal_write method of both LogWriter & ErrorLogger. Error should be caught...

Your checklist for this pull request

• I have reviewed the guidelines for contributing to this repository.
• I have added (or updated) appropriate tests if this PR fixes a bug or adds a feature.
• My pull request is 100 lines added/removed or less so that it can be easily reviewed.
• If this PR doesn't need tests (docs change), I added [ci skip] to the title of the PR.
• If this closes any issues, I have added "Closes #issue" to the PR description or my commit messages.
• I have updated the documentation accordingly.
• All new and existing tests passed, including Rubocop.

[vzajkov]

Thank you for this @MSP-Greg .

Is this alright to merge in? I believe this bug is preventing #2770 from passing checks.

[MSP-Greg]

@vzajkov

I don't think this is blocking, as the last failed CI job in #2700 was an intermittent failure. A rebase on master on your branch will force a new CI run.

Regardless, I'll merge this.