feature(h2c): optional certificates for TLS

[cv65kr]

Reason for This PR

In case of h2c there a no need to provide certificates with enabled TLS option

Description of Changes

If TLS is enabled certificates files are optional.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

• All commits in this PR are signed (git commit -s).
• The reason for this PR is clearly provided (issue no. or explanation).
• The description of changes is clear and encompassing.
• Any required documentation changes (code and docs) are included in this PR.
• Any user-facing changes are mentioned in CHANGELOG.md.
• All added/changed functionality is tested.

[rustatian]

Hey @cv65kr 👋
Thank you for the PR 👍
I see an error in the CI: https://github.com/temporalio/roadrunner-temporal/actions/runs/11846144712/job/33013137624?pr=579#step:15:9696

[rustatian]

I guess it is better to introduce an option in the TLS config, like, use_h2c: true/false. And if used, skip certificates checking and skip getClientCertificates method (just return nil based on the config).

[cv65kr]

@rustatian changed

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 40.06%. Comparing base (c5c2e9a) to head (8a838fa).
Report is 19 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #579   +/-   ##
=======================================
  Coverage   40.06%   40.06%           
=======================================
  Files          14       14           
  Lines         936      936           
=======================================
  Hits          375      375           
  Misses        518      518           
  Partials       43       43           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rustatian]

Thank you, @cv65kr 👍
I have a plan to release 2024.3 with that change this Thursday.