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

FEATURE REQUEST: Add a flag and feedback to clear all existing local records #33

Open
iamGBOX opened this issue Jun 10, 2021 · 6 comments · Fixed by #47
Open

FEATURE REQUEST: Add a flag and feedback to clear all existing local records #33

iamGBOX opened this issue Jun 10, 2021 · 6 comments · Fixed by #47
Assignees
Labels
enhancement New feature or request

Comments

@iamGBOX
Copy link

iamGBOX commented Jun 10, 2021

Detailed Description

It would be convenient to be able to use a flag on checkip to clear all existing records to check a set of addresses totally fresh, rather than running checkip -f or manually deleting the json records.

Context

This would permit a complete clear of all existing records from within the tool, which could streamline new investigations.

Possible Implementation

checkip -clear

Are you sure you want to clear ALL existing local records? (Y/N)

Local records cleared

Your Environment

  • Version used: Current
  • Operating System and version (desktop or mobile): macOS 11.4
@t-a-y-l-o-r t-a-y-l-o-r added the enhancement New feature or request label Jul 29, 2021
@t-a-y-l-o-r
Copy link
Owner

I could see an argument for adding this in as a flag like:

checkip --fresh -ip 8.8.8.8 would create a fresh record. Would behave similar to checkip --force -ip 8.8.8.8 but would also be a destructive action.

could also see:
checkip --no-record -ip 8.8.8.8 if you want to avoid adding this ip to the record if you know you scan it a lot.

Long term adding all this into the config is planned eventually. But currently have no timeline for that

@t-a-y-l-o-r t-a-y-l-o-r self-assigned this Aug 2, 2021
@iamGBOX
Copy link
Author

iamGBOX commented Aug 2, 2021

I think there's definite utility in both of those options, the, checkip --fresh -ip 8.8.8.8 and the checkip --no-record -ip 8.8.8.8, but I would also find the option to totally clear all local records useful, especially since the security status of records continues to become less relevant as they age.

@t-a-y-l-o-r
Copy link
Owner

t-a-y-l-o-r commented Aug 2, 2021

In practice are you asking for:

# note! no ip given
$ checkip --delete-cache

Which would delete all local records across the board. (i.e. the record and the report)

@iamGBOX
Copy link
Author

iamGBOX commented Aug 2, 2021

Yes! That's more what I was thinking of, an option to delete all local records across the board.

EDIT: A colleague suggested that I could use an alias in my shell to emulate this behaviour for now, but having that natively would be helpful.

@t-a-y-l-o-r
Copy link
Owner

sounds good. Won't be in this current set of fixes. But could easily be on the shortlist of new features after that

@t-a-y-l-o-r t-a-y-l-o-r linked a pull request Oct 31, 2021 that will close this issue
@t-a-y-l-o-r
Copy link
Owner

local testing went well. Just need confirmation from @iamGBOX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants