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

Reset DNS - add a warning #839

Closed
shoulders opened this issue Jun 26, 2024 · 6 comments
Closed

Reset DNS - add a warning #839

shoulders opened this issue Jun 26, 2024 · 6 comments

Comments

@shoulders
Copy link

SYSTEM INFORMATION
OS type and version Ubuntu Linux 22.04.4
Webmin version 2.111
Usermin version 2.010
Virtualmin version 7.10.0  
Theme version 21.10
Package updates 92 package updates are available

background

  • when I reset the DNS on virtual server it just immediately resets the DNS.
  • when i clicked on the reset dns button on a sub-server whoes DNS was being controlled by the parent, virtualmin warned me it was going to reset the DNS. The changes showed it was going to reset the parent DNS (and I assume the sub-servers which was part of it.

the issue

no warning message for normal virtual servers about what was going to be changed.

solution

add the prompt indicating what is going to be changed for normal virtual servers rather than immediately carrying out the command.

related

#838 Reset DNS button should not be available on sub-servers when parent controls the DNS

@jcameron
Copy link
Collaborator

Do you think it would be useful to show what DNS records are going to be added or removed?

@shoulders
Copy link
Author

On the sub server when I clicked the button it showed the records that were going to be removed, but to be honest I did not find that useful except that it showed me it was going to reset the parents zone so in this case it was.

Going forwards this 'bug' will not be an issue, so maybe a warning message will be fine.

Example message:

The DNS zone
can.example.com is going to be reset to default values as set by the relevant server template.

@jcameron
Copy link
Collaborator

Ok, this will be fixed in the next next Virtualmin release.

@shoulders
Copy link
Author

shoulders commented Jun 30, 2024

SYSTEM INFORMATION
OS type and version Ubuntu Linux 22.04.4
Webmin version 2.111
Usermin version 2.010
Virtualmin version 7.10.0  
Theme version 21.10
Package updates 94 package updates are available

I made some changes to the default server template and then using chrome on my dev virtualmin server, when i clicked the DNS reset button I got the following warning which is great but as you can see I am still on 7.10.0.

Do you only get this warning if a change has been noticed by Virtualmin?

image

I then made a change in my developer template on the dns page and then went and reset the dns again but this time the button went straight to action

image

@iliajie
Copy link
Collaborator

iliajie commented Jun 30, 2024

Related #774 (comment).

@jcameron
Copy link
Collaborator

Yeah the warning is only shown if some records are expected to be removed. Otherwise it proceeds without any warning.

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

No branches or pull requests

3 participants