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

Missing Combinations #5

Closed
jaikishantulswani opened this issue Feb 8, 2022 · 6 comments
Closed

Missing Combinations #5

jaikishantulswani opened this issue Feb 8, 2022 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@jaikishantulswani
Copy link

Hi @d0nutptr Here I create a new issue regarding missing combinations, missed in ripgen

domain
one-two.three.four-five.example.com

word
apple

Combinations missed by ripgen

apple-two.three.four-five.example.com
five-two.three.four-five.example.com
four-five-two.three.four-five.example.com
four-two.three.four-five.example.com
one-apple.three.four-five.example.com
one-five.three.four-five.example.com
one-four-five.three.four-five.example.com
one-four.three.four-five.example.com
one-one.three.four-five.example.com
one-three.three.four-five.example.com
three-two.three.four-five.example.com
two-two.three.four-five.example.com

Can you please check.

Thank you.

@d0nutptr
Copy link
Contributor

d0nutptr commented Feb 8, 2022

Hello!

This was a super simple fix :P I forgot a legacy word length filter in the swap generator. It should now respect the minimum word-length parameter that you pass into ripgen with -l or --len.

Again, you should be able to get the new update by doing cargo install --force ripgen.

When you run this example, you need to make sure that the word length is 3 or smaller so you pick up "one" (it defaults to filter out words less than 5 chars).

echo "one-two.three.four-five.example.com" | ripgen -l 3 -w words.txt | sort -u

@d0nutptr d0nutptr self-assigned this Feb 8, 2022
@d0nutptr d0nutptr added the bug Something isn't working label Feb 8, 2022
@d0nutptr
Copy link
Contributor

d0nutptr commented Feb 8, 2022

Side note: I should probably make a "unique" output flag :P

@jaikishantulswani
Copy link
Author

@d0nutptr Thank you so much for this fastest solution :) very helpful

@jaikishantulswani
Copy link
Author

jaikishantulswani commented Feb 8, 2022

@d0nutptr one more thing if possible, add a flag to skip numeric iterations or leave it on user to use a flag -numbers to use in case.

@d0nutptr
Copy link
Contributor

d0nutptr commented Feb 8, 2022

Unfortunately I'll have to work on this specific problem a bit later - if you could make a separate issue for this I can get back to it.

It's not as simple as just not using the numbers transform for rust reasons :^) so it'll require a couple of more changes to do effectively, though it's not terribly difficult - just busy for the rest of the night :)

@jaikishantulswani
Copy link
Author

@d0nutptr ok will raise a separate issue on this, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants