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

Add 'chomp slash' feature #60

Open
0xdevalias opened this issue Oct 7, 2017 · 1 comment
Open

Add 'chomp slash' feature #60

0xdevalias opened this issue Oct 7, 2017 · 1 comment

Comments

@0xdevalias
Copy link
Contributor

0xdevalias commented Oct 7, 2017

This is getting into a bit of an edge case territory, so it may not be best solved here.. but I was using one of the SecLists dictionaries and it prefixed the paths with /, which resulted in a // being generated by gobuster, which resulted in redirect behaviour on the website I was testing.

There is currently functionality to append a forward slash, I would see this as being the anti-case for that option.

-f	Append a forward-slash to each directory request (dir mode only)

It could probably be made more generic by allowing to append/chomp an arbitrary string (that defaults to /)

This may be easier (or at least nicer) to integrate into the CLI with #59

I would then see it being something similar to:

--append-slash
--append
--append /

--chomp-slash
--chomp
--chomp /

Edit: Re-reading this.. the current functionality is 'append', whereas I was thinking of 'prepend'. Maybe the current functionality of adding a slash could be controlled with a --prefix or --prepend and the current -f could be controlled with a --suffix or --append type flag.

@0xdevalias
Copy link
Contributor Author

Actually.. funnily enough, there already seems to be a PR that is related to this: #56

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

No branches or pull requests

1 participant