Closed
Conversation
* Print message after command prompt * Add results.go query for grabbing hostname IP addresses * Add command to print FQDN IPS via DNS * Add human readable option to command and correct typo in ip.go * Add delimiter flag * Add netname flag * Remove UUID and sort IPs * Handle no results and no FQDN specification * Update readme to include show-dns-fqdn-ips command * Change HostnameIPResults to IPResults to satisfy linter * Change variable f to fqdn
…f supplied with fewer than two arguments (#814)
* Add FqdnResult type in repository.go * Rename FqdnResult type to FQDNResult to fit convention * Add argument with command, no argument messages, and DB resource * Setup FQDN query * Add query for FQDNs * Handle case for no db or ip and print line * Print resolved FQDNs in non-human readable format * Adjust error handling to respond with same message regardless of num of arguments * Print raw results * Adjust show-dns-fqdn-ips to prompt user to supply database and FQDN if supplied with fewer than two arguments * Change header name in show-ip-dns-fqdns from Resolved FQDNs to Resolved FQDN * Change func names to fit convention suggested by make test * Change table header from Resolved FQDN to Queried FQDN * Adjust usage text to say IP Address instead of just IP --------- Co-authored-by: Logan Lembke <logo2462@gmail.com>
* Create copy of rita config file and replace localhost with db * Fix typo in Docker Usage guide * Update Docker Usage guide to assume user is not part of docker group
* Update ParseSubnets to account for IPv6 * Add unit test for TestFilterSingleIPv6 * Adusted error handling of the ip.go:ParseSubnets call chain * Add ParseNets test * Refactor import.go:run and add panic to ip.go:init * Add error handling test to TestParseSubnets * Improve TestParseSubnets readability * Refactor to make indentations consistent with style guidelines
Signed-off-by: guoguangwu <guoguangwu@magic-shield.com> Co-authored-by: Liza Tsibur <liza@activecountermeasures.com>
Co-Authored-By: Naomi Kramer <naomiagoddard@gmail.com>
Co-Authored-By: Naomi Kramer <naomiagoddard@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.