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

Update exposed-kibana.yaml #7657

Merged
merged 3 commits into from
Jul 12, 2023
Merged

Update exposed-kibana.yaml #7657

merged 3 commits into from
Jul 12, 2023

Conversation

dongpohezui
Copy link
Contributor

Template / PR Information

  • add a path in requests
  • because some versions of Kibana do not include "cluster_uuid", delete a word in matchers

Template Validation

I've validated this template locally?

  • YES
  • NO

@DhiyaneshGeek DhiyaneshGeek self-assigned this Jul 11, 2023
@DhiyaneshGeek DhiyaneshGeek added the Done Ready to merge label Jul 11, 2023
Copy link
Contributor Author

@dongpohezui dongpohezui left a comment

Choose a reason for hiding this comment

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

'{{BaseURL}}/app/kibana' and '{{BaseURL}}/app/kibana/' are different !

@pussycat0x pussycat0x added the good first issue Good for newcomers label Jul 12, 2023
@DhiyaneshGeek
Copy link
Member

Hi @dongpohezui thank you so much for updating this template and contributing to this project 🍻

You can join our discord server. It's a great place to connect with fellow contributors and stay updated with the latest developments. Thank you once again

@DhiyaneshGeek DhiyaneshGeek merged commit 4d44db7 into projectdiscovery:main Jul 12, 2023
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 good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants