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 Pentest Cheat Sheets #3068

Closed
wants to merge 2 commits into from
Closed

Conversation

ByteSnipers
Copy link

Added Awesome Pentest Cheat Sheets

URL: https://github.com/ByteSnipers/awesome-pentest-cheat-sheets

Awesome Pentest Cheat Sheets contains a lot of security and pentesting related information. The list has a huge list of very sorted and selected resources, which can save a lot of time to any security oriented people. This list was started 2017 and was improved over the years. The initial awesome pentest cheat sheets collects over 3.8k stars

Added link to awesome-pentest-cheat-sheets
@sindresorhus
Copy link
Owner

Thanks for making an Awesome list! 🙌

It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

@Polycarbohydrate
Copy link
Contributor

Polycarbohydrate commented Jun 14, 2024

It appears that many of your items do not have any description. You also need to review at least 2 pull requests before making your own pull request.

Additionally running the linter on your repo, the following errors are given:

  README.md:323:3
  ×    9:1    ToC item "Security Talks and Videos" does not match corresponding heading "🖥 Security Talks and  Videos"                                            remark-lint:awesome-toc
  ×    9:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-security-talks-and-videos                                                            remark-lint:double-link
  ×   11:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-mobile-pentesting                                                                    remark-lint:double-link
  ×   12:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#%EF%B8%8F-cloud-pentesting                                                            remark-lint:double-link
  ×   13:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#active-directory                                                                      remark-lint:double-link
  ×   14:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-pentest-methodology                                                                  remark-lint:double-link
  ×   15:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-privilege-escalation                                                                 remark-lint:double-link
  ×   16:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#%EF%B8%8F-web-pentesting                                                              remark-lint:double-link
  ×   17:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#binary-exploitation                                                                   remark-lint:double-link
  ×   18:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#learning-platforms                                                                    remark-lint:double-link
  ×   19:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#bug-bounty                                                                            remark-lint:double-link
  ×   20:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#tools                                                                                 remark-lint:double-link
  ×   21:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/?tab=readme-ov-file#payloads-1                                                         remark-lint:double-link
  ×   22:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/?tab=readme-ov-file#programming                                                        remark-lint:double-link
  ×   60:100  Unexpected reference to undefined definition, expected corresponding definition (pdf) for a link or escaped opening bracket (\[) for regular text  remark-lint:no-undefined-references
  ×  243:3    https://owasp.org/www-project-vulnerable-web-applications-directory/#div-online                                                                    remark-lint:double-link
  ×  251:3    https://owasp.org/www-project-vulnerable-web-applications-directory/#div-offline                                                                   remark-lint:double-link
  ×  323:3    Don’t pad link with inner spaces                                                                                                                   remark-lint:no-inline-padding

  19 errors

For the first error just make your title for the header 🖥 Security Talks and Videos the same by either removing the emoji from the header or adding the emoji to the title in the Table of Contents(ToC).

The double-link errors can be easily solved by changing the links to #(Your header here). Example: https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-security-talks-and-videos ---> #-security-talks-and-videos.

Then the last issue needs to be fixed too. (no-inline-padding)

@Polycarbohydrate Polycarbohydrate mentioned this pull request Jun 14, 2024
readme.md Outdated
@@ -650,6 +650,7 @@
## Security

- [Application Security](https://github.com/paragonie/awesome-appsec#readme)
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets)
Copy link
Contributor

Choose a reason for hiding this comment

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

The URL to your list should end in #readme.

Suggested change
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets)
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets#readme)

@IAmCoder IAmCoder mentioned this pull request Jun 24, 2024
33 tasks
@ByteSnipers
Copy link
Author

I've update the repository, but I still have some issues, what I cannot solve, because I cannot identify why they are issues.

✖ 15:1 ToC item "Privilege Escalation" does not match corresponding heading "Web Pentesting" remark-lint:awesome-toc
✖ 243:3 https://owasp.org/www-project-vulnerable-web-applications-directory/#div-online remark-lint:double-link
✖ 251:3 https://owasp.org/www-project-vulnerable-web-applications-directory/#div-offline remark-lint:double-link
✖ 323:3 Don’t pad link with inner spaces remark-lint:no-inline-padding

@tyaga001
Copy link
Contributor

tyaga001 commented Jul 3, 2024

I run your repo in Linting and can see some errors are still need to be resolved.
image

also you have no pull requests reviews links so I suggest you reading through it one more time! :)

cc. @sindresorhus

@tyaga001 tyaga001 mentioned this pull request Jul 3, 2024
33 tasks
@0xAnalyst 0xAnalyst mentioned this pull request Jul 14, 2024
3 tasks
@0xAnalyst
Copy link

Apart from fixing liniting issues I suggest you add a books sections and books related to book bountry web penetration testing mobile app pentesting....etc

Copy link

@0xAnalyst 0xAnalyst left a comment

Choose a reason for hiding this comment

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

Add it to the end of the list

@hstsethi
Copy link

hstsethi commented Jul 24, 2024

This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

As per the guidelines, the title of the pull request should be "Add Pentest Cheat Sheets".

@ByteSnipers ByteSnipers changed the title Add Awesome Pentest Cheat Sheets Add Pentest Cheat Sheets Jul 24, 2024
@sindresorhus sindresorhus force-pushed the main branch 5 times, most recently from f0658c5 to 993cee4 Compare August 8, 2024 13:04
@0xAnalyst 0xAnalyst mentioned this pull request Aug 11, 2024
3 tasks
@hstsethi hstsethi mentioned this pull request Aug 25, 2024
33 tasks
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.

7 participants