Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Sortlist enabled by default #961

Closed
wants to merge 2 commits into from

4 participants

@jbahillo
Collaborator

No description provided.

@jacalvo
Owner

Changelog entry missing

@jbahillo
Collaborator

Added, sorry ;)

@jacalvo
Owner

Jenkins retest this please.

@sixstone-qq
Owner

Don't you think this is a change in the behaviour that should go to latest community version?

@jbahillo
Collaborator

AFAIK, sortlist is enabled by default in master and 3.3 branches. This is jus a suggestion to do this as well for 3.2, as it is knwon that non having it usually creates issues with samba module

@jbahillo
Collaborator

Jenkins retest this please.

@papajulio
Owner

Jenkins retest this please.

@jbahillo
Collaborator

Jenkins retest this please.

@jacalvo
Owner

If we haven't merged this yet (as there is no consensus, because I think like Quique, this is a change of behavior in 3.2), I don't think now is the moment to do it. I'm closing this, but don't hesitate to reopen if you have any strong reason to do it.

@jacalvo jacalvo closed this
@jbahillo jbahillo deleted the jbahillo/sortlist-default branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 10, 2014
  1. @jbahillo

    Sortlist enabled by default

    jbahillo authored
Commits on Feb 11, 2014
  1. @jbahillo
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 2 deletions.
  1. +2 −0  main/dns/ChangeLog
  2. +2 −2 main/dns/conf/dns.conf
View
2  main/dns/ChangeLog
@@ -1,3 +1,5 @@
+HEAD
+ + Sortlist is enabled by default
3.2.5
+ Lowercase LDB domains for comparision
+ Update TXT _kerberos record on domain rename
View
4 main/dns/conf/dns.conf
@@ -25,5 +25,5 @@ intnets =
generate_reverse_zones = yes
# This key defines whether you want to sort the results based on the querying IP
-# Uncomment it to enable it
-# sortlist = yes
+# Comment it to disable it
+sortlist = yes
Something went wrong with that request. Please try again.