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

Improve generic-xxe.yaml #10359

Merged
merged 3 commits into from
Jul 25, 2024
Merged

Improve generic-xxe.yaml #10359

merged 3 commits into from
Jul 25, 2024

Conversation

Ahsraeisi
Copy link
Contributor

Template / PR Information

  • Added OOB testing for XXE
  • Removed "OR" matcher condition, because it was not necessary (it's OR by default)

Template Validation

I've validated this template locally?

  • YES
  • NO

@DhiyaneshGeek DhiyaneshGeek self-assigned this Jul 24, 2024
@DhiyaneshGeek DhiyaneshGeek added the Done Ready to merge label Jul 24, 2024
@DhiyaneshGeek DhiyaneshGeek requested a review from pussycat0x July 24, 2024 08:33
@DhiyaneshGeek
Copy link
Member

Hi @Ahsraeisi

i have update the author name with your github username , does that work for you ?

Looking forward to hear back from you

Thanks

@Ahsraeisi
Copy link
Contributor Author

Tnx for adding my name. I forgot it.
I change it to my desired name.

@DhiyaneshGeek
Copy link
Member

Hi @Ahsraeisi

Thanks for letting us know

@pussycat0x
Copy link
Contributor

pussycat0x commented Jul 25, 2024

Hi @Ahsraeisi Thank you for updating this template with the community and for your contribution to this project. Your efforts are greatly appreciated. Cheers!

@pussycat0x pussycat0x merged commit c6221aa into projectdiscovery:main Jul 25, 2024
2 checks passed
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.

3 participants