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

Lowercase query options #11

Closed
tk-nguyen opened this issue Jan 12, 2021 · 3 comments · Fixed by #14
Closed

Lowercase query options #11

tk-nguyen opened this issue Jan 12, 2021 · 3 comments · Fixed by #14

Comments

@tk-nguyen
Copy link

Hello, I've been using this tool for several days and I'm loving it so far!
However, there's one minor issue: the tool does not recognize query types in lowercase (a instead of A, txt instead of TXT, mx instead of MX, etc...). I'm not familiar with C# per se, but I think this can be fixed by converting all query types to uppercase.
It would be better if dug behaved like dig, which is parsing query types when they're specified in place of the "verbs" i.e. dug mx example.com, since dug by default run with the verb "run".
Thank you for creating this tool!

@MattInternet
Copy link
Contributor

I think the second part of this ticket should be broken into another issue. Im not sure it will be very straightforward with the way we are parsing the args, but it should be looked into!

@MattInternet
Copy link
Contributor

Hey @tk-nguyen , I created #15 to capture your second idea. Also thanks for the issue! :)

@tk-nguyen
Copy link
Author

Thanks for being so quick in fixing it 👍 !

This issue was closed.
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

Successfully merging a pull request may close this issue.

2 participants