Skip to content

Conversation

@jfbes
Copy link
Contributor

@jfbes jfbes commented Nov 30, 2023

Template / PR Information

This detects prometheus exposed consoles in different endpoints than the previous template detecting prometheus

  • References:

Template Validation

I've validated this template locally?

  • YES
  • NO

Additional Details (leave it blank if not applicable)

Additional References:

@brigancc
Copy link

This could be rolled into the main template for this by adding the second path - "{{BaseURL}}/prometheus/graph" as another kind-of-default or easy-to-guess place to check. This is based on actual feedback on implementations where we have seen prometheus deployed.

@DhiyaneshGeek DhiyaneshGeek self-assigned this Dec 1, 2023
@DhiyaneshGeek
Copy link
Member

Hi @jfbes i have added the additional endpoint in the old template and add you as co-author for the same

Let me know if the changes looks good

Looking forward to hear back from you 😄

Thanks
@DhiyaneshGeek

@DhiyaneshGeek DhiyaneshGeek added the Done Ready to merge label Dec 1, 2023
@DhiyaneshGeek DhiyaneshGeek changed the title Create prometheus-exposed-panel2.yaml Create prometheus-exposed-panel.yaml Dec 1, 2023
@DhiyaneshGeek DhiyaneshGeek added the good first issue Good for newcomers label Dec 1, 2023
@ritikchaddha ritikchaddha merged commit 06e4473 into projectdiscovery:main Dec 2, 2023
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.

4 participants