-
Notifications
You must be signed in to change notification settings - Fork 357
bumping patch version for DB hotfix #832
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
Open
SamuelCarroll
wants to merge
20
commits into
develop
Choose a base branch
from
2326-CVE-Fix
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
* 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 <[email protected]>
* 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 <[email protected]> Co-authored-by: Liza Tsibur <[email protected]>
Co-Authored-By: Naomi Kramer <[email protected]>
Co-Authored-By: Naomi Kramer <[email protected]>
0x6d6f7468
requested changes
Jan 9, 2026
0x6d6f7468
reviewed
Jan 9, 2026
Contributor
Author
|
Tried merging with develop to solve issues, lots of things pop up as changes as a result... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Want to add a new binary in addition to this, would allow AC-Hunter change to just pull latest release/binary