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

ffuf: add page #6183

Merged
merged 14 commits into from
Jul 15, 2021
Merged

ffuf: add page #6183

merged 14 commits into from
Jul 15, 2021

Conversation

otterbaub
Copy link
Contributor

@otterbaub otterbaub commented Jun 29, 2021

Request to add FUFF.md to TLDR
Ran .md against tldr-lint , changed EOL from WIN to UNIX, No other error on re-run.
Cheers

  • The page (if new), does not already exist in the repo.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

Request to add FUFF.md to TLDR
Ran .md against tldr-lint , changed EOL from WIN to UNIX, No other error on re-run.
Cheers
@CleanMachine1 CleanMachine1 changed the title Create fuff.md fuff: add page Jun 29, 2021
@CleanMachine1 CleanMachine1 added the new command Issues requesting creation of a new page. label Jun 29, 2021
@CleanMachine1
Copy link
Member

Thanks for your commit. Please make sure to read the guidelines to get the title format.

You need to stick with during the process of getting your paged approved.

This will take a minimum of 24-48 hours so that all members and collaborators have enough time to respond.

@CleanMachine1
Copy link
Member

CleanMachine1 commented Jun 29, 2021

This command is ffuf not fuff. We need to change the entire file name

@CleanMachine1
Copy link
Member

CleanMachine1 commented Jun 29, 2021

And it is a common command, it supports Mac, Windows and Linux (even BSD).

@otterbaub otterbaub changed the title fuff: add page ffuf: add page Jun 29, 2021
changed file name fuff > ffuf
fixed typo in command name
pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
Replaced line 3 with updated description "Subdomain and directory discovery tool."
Removed line 8,16,20,24 leading forward-slash
Removed line 12 trailing forward-slash on usl
pages/linux/ffuf.md Outdated Show resolved Hide resolved
@sbrl
Copy link
Member

sbrl commented Jul 2, 2021

@CleanMachine1: If the title doesn't conform to our convention, feel free to just edit it. That way we avoid additional friction for contributors.

Line 6 added "output" description
Line 26,28 added a new description/command to reflect the program description.
Line 6,10,14,18,22,26  Modifed description to be in the the imperative stance.
@tldr-bot

This comment has been minimized.

Repalce 10,18,22 Fuzzing > Fuzz
Copy link
Contributor Author

@otterbaub otterbaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed fuzzing to fuzz

Copy link
Contributor Author

@otterbaub otterbaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

made changes inline with automated check

pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
edited line 28, placed double quotes outside of curly bracket indicator.
Line 6,10,14,18,22 added square brackets around first letter of keyword indicators that correspond with requisite program argument flag,  as an indicator of use
pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
otterbaub and others added 2 commits July 7, 2021 12:43
Co-authored-by: Axel Navarro <[email protected]>
Co-authored-by: Axel Navarro <[email protected]>
pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
pages/linux/ffuf.md Outdated Show resolved Hide resolved
otterbaub and others added 3 commits July 8, 2021 09:46
Co-authored-by: marchersimon <50295997 [email protected]>
Co-authored-by: marchersimon <50295997 [email protected]>
Co-authored-by: marchersimon <50295997 [email protected]>
Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @otterbaub

pages/linux/ffuf.md Outdated Show resolved Hide resolved
Line 10 swapped capitalization to reflect actual flag utilization.
> Subdomain and directory discovery tool.
> More information: <https://github.com/ffuf/ffuf>.

- Discover directories using a [w]ordlist on a target [u]rl with [c]olorized and [v]erbose ouput:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't mean that there was any error with the letters for the flags being encased. I just personally dislike them, however if everyone else thinks they are fine then feel free to merge

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm also not a fan, but I think that should be discussd in a dedicated issue.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CleanMachine1 the PR got too long, the best was to merge it, and now we can refine the page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new command Issues requesting creation of a new page.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants