Skip to content

DNSSEC improvements#1724

Closed
xsuchy wants to merge 2 commits intorpm-software-management:masterfrom
xsuchy:dnssec1
Closed

DNSSEC improvements#1724
xsuchy wants to merge 2 commits intorpm-software-management:masterfrom
xsuchy:dnssec1

Conversation

@xsuchy
Copy link
Copy Markdown
Member

@xsuchy xsuchy commented Feb 10, 2021

Better representation of KeyInfo() to easy debugging.
Disable local resolve.conf for DNSSEC query (see commit).

@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Feb 10, 2021

Hello @xsuchy! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-02-10 16:22:22 UTC

because it use systemd-resolved by default, and it cannot do DNSSEC validation
systemd/systemd#4621

Commenting out instead of deleting, to remind us that sometimes in future we can return it back.
Hopefully.
@pemensik
Copy link
Copy Markdown
Contributor

Could disabling of resolvconf usage be configurable by config option? There are multiple working local caches, which allow DNSSEC validation just fine. There is no reason to turn off resolver usage for dnssec-trigger user for example.

Sadly, systemd-resolved (current default) seems to be the only implementation, which always prevents resolver usage.

@xsuchy
Copy link
Copy Markdown
Member Author

xsuchy commented Feb 12, 2021

As long as it is the default Fedora configuration, I would say yes. My $0.02

@xsuchy
Copy link
Copy Markdown
Member Author

xsuchy commented Jun 24, 2021

This is working now and not needed anymore.

@xsuchy xsuchy closed this Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants