Skip to content

Conversation

@dentarg
Copy link
Member

@dentarg dentarg commented Dec 11, 2021

Close #2708
Close #2711

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.

@dentarg dentarg force-pushed the ssl_bind-localhost-self-signed-fix branch from 6fba34a to 3f0ba58 Compare December 12, 2021 11:59
@dentarg
Copy link
Member Author

dentarg commented Dec 12, 2021

Added missing skip_if :jruby to test_ssl_self_signed_configuration_from_DSL

@dentarg
Copy link
Member Author

dentarg commented Dec 12, 2021

macos-10.15 fail on ruby 2.2 was a strange one though 😕 Gem::RuntimeRequirementNotMetError: json requires Ruby version >= 2.3. but it worked on macos-11 with ruby 2.2 what is going on 🤯

@dentarg dentarg force-pushed the ssl_bind-localhost-self-signed-fix branch from 3f0ba58 to f2b573c Compare December 12, 2021 20:33
@MSP-Greg
Copy link
Member

@dentarg Again, working on intermittent CI issues. macos-10.15 just does weird stuff. With (yet another) CI patch, I haven't had failures/errors except with 10.15...

Close puma#2708
Close puma#2711

Co-authored-by: Marcin Olichwirowicz <[email protected]>
@dentarg dentarg force-pushed the ssl_bind-localhost-self-signed-fix branch from f2b573c to 2b75c64 Compare December 13, 2021 00:07
@dentarg dentarg requested a review from MSP-Greg December 13, 2021 00:17
@dentarg
Copy link
Member Author

dentarg commented Dec 14, 2021

@MSP-Greg @nateberkopec WDYT?

@dentarg
Copy link
Member Author

dentarg commented Jan 5, 2022

@nateberkopec could be included in next release?

@nateberkopec nateberkopec added the waiting-for-review Waiting on review from anyone label Jan 9, 2022
@nateberkopec nateberkopec merged commit 03be0aa into puma:master Jan 18, 2022
@nateberkopec
Copy link
Member

Cheers @dentarg! Thanks for your patience.

@dentarg dentarg deleted the ssl_bind-localhost-self-signed-fix branch January 18, 2022 19:09
dentarg added a commit to Starkast/wikimum that referenced this pull request Jan 27, 2022
No need to care about the port gotchas with foreman now that we can use
ssl_bind in the Puma config: puma/puma#2764

Close #309
JuanitoFatas pushed a commit to JuanitoFatas/puma that referenced this pull request Sep 9, 2022
Close puma#2708
Close puma#2711

Co-authored-by: Marcin Olichwirowicz <[email protected]>

Co-authored-by: Marcin Olichwirowicz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature waiting-for-review Waiting on review from anyone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ssl_bind not compatible with localhost integration

3 participants