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

Allow use in production #163

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

LiamLoewen
Copy link

[FEATURE] Allow use in production

Issue or feature explanation

Providing the option to access Prequel in production will allow it to be used in system administrator dashboards for inexpensive DB operations. Setting it behind a warning and defaulting it to false will discourage its use for those who don't know exactly what they are doing.

Proposed solution/change

An additional boolean config setting (default: false) which can be enabled from .env was added to config/prequel.php. This variable is then checked in the Authorised middleware, allowing the configuration check to return true for enabled when the app environment is production and the production_enabled flag is enabled.

Love this package by the way, it's so much better than phpMyAdmin.

Added another config boolean to allow access in production environments.
@QuintenJustus
Copy link
Member

Nice one! Thanks for contributing, will include in next release.

@pratik149
Copy link

Along with this, better if we also have a "Basic HTTP Auth" to protect it in production.

I have seen this implemented in Log Viewer package.

https://github.com/opcodesio/log-viewer/blob/f5ac2dffd6ca734e54995807d12cef5db7304940/config/log-viewer.php#L107

@sinceow
Copy link

sinceow commented Apr 1, 2024

Looking forward to ...

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

Successfully merging this pull request may close these issues.

None yet

4 participants