Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not try to sign certificates for None if ip not resolved #733

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tbaumann
Copy link

Fixes #732

Tilman Baumann added 2 commits September 14, 2022 18:13
Filling in local hostname if a local IP can't be resolved seems like a
better fallback.
Revisiting 4231a52

Also ignoring errors when trying to look up a IPv6 address in a IPv4
subnet which will not work.

Referring to  juju#732
@ajkavanagh
Copy link
Contributor

It looks like tests are failing: tox -e py310 (that uses nose2):

----------------------------------------------------------------------
Ran 1936 tests in 7.439s

FAILED (errors=5)
Name                          

@ajkavanagh ajkavanagh added the Incomplete For an issue, needs more information. For a PR, needs further work. label May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Incomplete For an issue, needs more information. For a PR, needs further work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

contrib.opensack.cert_utils.get_certificate_request like() can greate None indexted list
2 participants