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

Feature-Request: allow-notify-from as per-zone setting #8816

Open
mjaepel opened this issue Feb 12, 2020 · 4 comments
Open

Feature-Request: allow-notify-from as per-zone setting #8816

mjaepel opened this issue Feb 12, 2020 · 4 comments

Comments

@mjaepel
Copy link

mjaepel commented Feb 12, 2020

  • Program: Authoritative
  • Issue type: Feature request

Short description

There is a global setting for allow-notify-from in authoritative server settings. But it's not available as per-domain setting in domain metadata.
Please implement it there, too. So it should be consistent with allow-axfr-ips (global) / allow-axfr-from (per-zone)

Usecase

Some special dns setups use different servers / source ips for notifies and axfr. Currently the only way to allow this scenario is to configure the notify IPs globally. But so they could send notifies for all configured domains.
In multi-tenant / shared environments you wouldn't allow an other tenant to notify domains eachother. ;)

Description

It's so simple to explain. See above. :)

@Habbie
Copy link
Member

Habbie commented Feb 12, 2020

In multi-tenant / shared environments you wouldn't allow an other tenant to notify domains eachother. ;)

This is a serious question: why not?

@mjaepel
Copy link
Author

mjaepel commented Feb 13, 2020

It's one way (of many) to start a DoS attack against DNS servers.
Other reason is that some customers want to control their notifys by their self and not by others. If you plan a big maintenance you want to schedule notifies by your own. It's crucial for success if you send the notify in the correct moment or any one else disrupt your maintenance with a single notify at the wrong time.

Another technically reason is the management of allowed notify ips. If you have thousands domains in one instance you will also have thousand different notify source ips. It's difficult to managed this in only one option.
It's much easier to managed per domain because there will be only a small count of notify ips per domain.

The feature is already implemented globally. So I hope we haven't discuss the sense of the feature itself.
So it would be nice when this feature is adjustable by domain like axfr.

@Habbie Habbie added this to the auth-helpneeded milestone Feb 13, 2020
@Habbie
Copy link
Member

Habbie commented Feb 13, 2020

It's crucial for success if you send the notify in the correct moment or any one else disrupt your maintenance with a single notify at the wrong time.

This does not make sense. If this is true, you already had way bigger problems.

It's much easier to managed per domain because there will be only a small count of notify ips per domain.

But I like this reason.

I have put your request in the 'auth-helpneeded' milestone, which means that we think it's a decent idea, but will not put it on the roadmap for now. If somebody submits a good patch for it, we will merge it.

@aj-gh
Copy link
Contributor

aj-gh commented Dec 29, 2021

+1
Having this as zone metadata would allow it to be manipulated by non-admins via the API instead of having to add potentially untrusted third party IP addresses into global configuration. Luckily that setup is quite rare otherwise that list might become huge.
As a workaround the notify IP could be added as additional primary but that has the drawback that about a half of all SOA checks will fail unless that IP is also responding to these checks (which is likely not the case otherwise it could be a real primary in the first place).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants