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.

Sign up for GitHub

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

Jump to bottom

Remove feature "Randomized capitalization" #160

Open
matsduf opened this issue Nov 27, 2022 · 4 comments · Fixed by #206 · May be fixed by #207
Open

Remove feature "Randomized capitalization" #160

matsduf opened this issue Nov 27, 2022 · 4 comments · Fixed by #206 · May be fixed by #207
Labels
S-PRforIssue Status: There is a PR that is meant to resolve the issue
Milestone
v2024.2

Comments

@matsduf
Copy link
Contributor

matsduf commented Nov 27, 2022

Zonemaster-LDNS is mean to for Zonemaster-Engine (and Zonemaster-Backend and Zonemaster-CLI). The feature "Randomized capitalization" is not useful for Zonemaster and there is no reason to believe it will be.

Make the the feature deperacted from v2023.1 and removed from v2023.2.
@matsduf matsduf added this to the v2023.1 milestone Nov 27, 2022
@matsduf
Copy link
Contributor Author

matsduf commented Mar 9, 2023

@pawal, do you have any insight in the randomization feature? It is off by default in Zonemaster.

@ghost
Copy link

ghost commented Mar 9, 2023

It seems it was added with 77ca58f7 and dca14a4

@matsduf matsduf modified the milestones: v2023.1, v2023.2 Jun 26, 2023
@matsduf matsduf modified the milestones: v2023.2, v2024.1 Mar 19, 2024
@matsduf matsduf modified the milestones: v2024.1, v2024.2 Jul 3, 2024
@marc-vanderwal
Copy link
Contributor

Shall we deprecate this feature now and remove it in 2025.1?

@matsduf
Copy link
Contributor Author

matsduf commented Sep 2, 2024

Yes, that is a good suggestion.

This was referenced Sep 2, 2024
Merged
Draft
@marc-vanderwal marc-vanderwal added the S-PRforIssue Status: There is a PR that is meant to resolve the issue label Sep 3, 2024
marc-vanderwal added a commit that referenced this issue Sep 5, 2024
@marc-vanderwal
Merge pull request #206 from marc-vanderwal/feature/#160/part-1
17b3a80 
Deprecate case randomization feature
This was linked to pull requests Sep 12, 2024
[DO NOT MERGE YET] Remove feature "randomized capitalization" #207
Draft
Deprecate case randomization feature #206
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
S-PRforIssue Status: There is a PR that is meant to resolve the issue
Projects
None yet
Milestone
v2024.2
2 participants
@matsduf @marc-vanderwal