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

Added template for unigui-server-monitor-exposure #9746

Merged
merged 3 commits into from May 12, 2024

Conversation

ehsandeep
Copy link
Member

Added template for unigui-server-monitor-exposure

@ritikchaddha ritikchaddha added the Done Ready to merge label May 10, 2024
@serrapa
Copy link
Contributor

serrapa commented May 10, 2024

Hey @ritikchaddha thanks for the fixes! I just used the AI Assistant on the Template Editor to create the template. Such an amazing feature!

I wonder, why did you change the severity from medium to low? In some cases when there is a particolar flag in the configuration (as said in the docs here: https://unigui.com/resources/online-documentation/developer-manual), users sessions tokens are listed. Even more you can shutdown the server (critical functions under the "Manage" tab) in the default configuration.

image

All of these things are possible if the authentication on the endpoint is not enabled and it's not by default.

@ritikchaddha
Copy link
Contributor

ritikchaddha commented May 10, 2024

Hi there, @serrapa We appreciate your contribution to this project and sharing this template with the community. 🍻
But when I tested this template on a couple of vulnerable hosts, none of them had the Manage option or exposed user session tokens. This means that it must only be exposing on a small number of vulnerable hosts, but I haven't yet come across any. So in that case we're keeping the severity low.

…iguration/unigui-server-monitor-exposure.yaml
@DhiyaneshGeek DhiyaneshGeek merged commit 787dffe into main May 12, 2024
2 checks passed
@DhiyaneshGeek DhiyaneshGeek deleted the Paolo-Serra-3d8bAU7Fq6Pp8JQq4gdpWx branch May 12, 2024 08:12
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.

None yet

4 participants