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

Modified CVE-2024-7928.yaml to extract data based on regex #10588

Closed
wants to merge 4 commits into from

Conversation

Sicks3c
Copy link
Contributor

@Sicks3c Sicks3c commented Aug 22, 2024

Template / PR Information

I have modified the CVE-2024-7928.yaml in order to use extractors to grab the necessary values form the response

Template Validation

Screenshot 2024-08-22 at 13 22 28

I've validated this template locally?

  • YES
  • NO

@DhiyaneshGeek DhiyaneshGeek self-assigned this Aug 23, 2024
@DhiyaneshGeek
Copy link
Member

Hi @Sicks3c

if we print the entire jsonpReturn data , it will print the entire output in extractor format

i have marked it for further review and will keep you posted

Thank you

@DhiyaneshGeek DhiyaneshGeek added Status: Review Needed The issue has a PR attached to it which needs to be reviewed and removed waiting for more info labels Aug 23, 2024
@Sicks3c
Copy link
Contributor Author

Sicks3c commented Aug 24, 2024

Hello @DhiyaneshGeek

Thank you for your comment, that's exactly the point it will help to view the data in the output

Cheers

@ritikchaddha
Copy link
Contributor

Hello @Sicks3c,

Thank you for updating the template. However, we do not add the extractor to print the entire response in the output, which made the output data unclear to the user. Try to add an extractor that extracts limited data, such as API keys and versions.

If you want to see the response data, you can use the -debug-resp option.

@ritikchaddha ritikchaddha added waiting for more info and removed Status: Review Needed The issue has a PR attached to it which needs to be reviewed labels Aug 27, 2024
@DhiyaneshGeek
Copy link
Member

Hi @Sicks3c

due to inactivity we are closing this PR, fell free raise another PR with more information

@DhiyaneshGeek DhiyaneshGeek added Done Ready to merge and removed waiting for more info labels Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done Ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants