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

Added match alias only option & some other things #15

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

distahl
Copy link

@distahl distahl commented May 11, 2021

  • Added an option (--aliases-only) to match only the alias against pattern. (4adfa05 & b9d54c0)
    • And on line 913 I replaced &re with &exclude_re. I think this was a bug.
  • Added additional regex targets, which can help when using --aliases-only (be1a656)
  • I ran into an issue (segfault) were a Cisco ASA was reporting less interfaces then needed. Tried to fix that with 803f844 +
    43f7b07 + df2ce93
  • I ran into an issue (segfault) within debug mode. I think I fixed that with e19fa31

Commits: 4adfa05 & b9d54c0 & be1a656 & 803f844 & 43f7b07 & df2ce93 & e19fa31

@distahl distahl changed the title Added match alias only option Added match alias only option & some other things May 27, 2021
@martialblog
Copy link
Member

This PR needs to be rebased after the refactor.

@RincewindsHat Can you please evaluate the proposed changes. Thanks!

@RincewindsHat
Copy link
Member

Rebase won't do this. But yes, it's in my scopes

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 this pull request may close these issues.

3 participants