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

lowercase-outgoing: port to 4.x, test, document #3441

Closed
Habbie opened this issue Feb 24, 2016 · 5 comments
Closed

lowercase-outgoing: port to 4.x, test, document #3441

Habbie opened this issue Feb 24, 2016 · 5 comments
Labels
rec
Milestone
rec-4.0.0

Comments

@Habbie
Copy link
Member

Habbie commented Feb 24, 2016

As merged without docs and tests in #3350.
@Habbie
Copy link
Member Author

Habbie commented Feb 24, 2016

Also (I thought we had a separate ticket but I can't find it), port this to 4.x.

@zeha zeha added the rec label Mar 7, 2016
@zeha zeha changed the title document, test lowercase-outgoing feature lowercase-outgoing: port to 4.x, test, document Mar 7, 2016
@pieterlexis pieterlexis added this to the rec-4.0.0 milestone Mar 23, 2016
@pieterlexis
Copy link
Contributor

How do you propose we test this? Running 2 recursors, one with this setting one without and letting the auth print the query it got from the recursor?

@Habbie
Copy link
Member Author

Habbie commented Mar 23, 2016

@Habbie
Copy link
Member Author

Habbie commented Mar 23, 2016

I don't have a better idea :)

@Habbie
Copy link
Member Author

Habbie commented Mar 23, 2016

It looks like our recursor testing is going to involve multiple recursors anyway, for various reasons :)

@pieterlexis pieterlexis mentioned this issue Mar 23, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
rec
Projects
None yet
Milestone
rec-4.0.0
Development

No branches or pull requests
3 participants
@zeha @Habbie @pieterlexis