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

Information disclosure of highly sensible information #141

Open
tpetry opened this issue Dec 14, 2018 · 1 comment
Open

Information disclosure of highly sensible information #141

tpetry opened this issue Dec 14, 2018 · 1 comment

Comments

@tpetry
Copy link

tpetry commented Dec 14, 2018

At the moment the code is exporting the complete configuration of all health resource to the ui. This means some highly sensitive information is exported, like the APP_KEY (used for encryption), database or external service credentials. But there's absolutely no need to export anything of it. The code should be cleaned to only export relevant information for the ui and never ever (!) export any of the resource configuration options as they may contain very sensitive information. They should never be transmitted!

One could argue that the health panel should only be visible to administrators and route middleware should be configured to ensure this, but...

  • Most people will use it without further configuration. I mean who would expect his database credentials embedded in a health page.
  • If no HTTPs is used this information is sent in plaintext on the web
  • This information may be cached by the browser. I hope you are never opening it from anywhere than your devices.
  • There may be many more reasons.

At the moment using this plugin is in my humble opinion a very hard security breach.

@joveice
Copy link

joveice commented Sep 30, 2019

I haven't even thought about this, this is enough for me to not use it until it's changed. The project is really good, but I can't have this security flaw.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants