loadsys / loadsys/CakePHP-Skeleton

_pi.php potentially a security risk?

Open
#131 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement question
Dominant language
PHP
Stars
10
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Now that this skeleton is open source, it's conceivable that someone might discover our trick for "protecting" the default phpinfo page we bundle.

One option might be to make it a .template file, and have someone setting up a new project write a "unique" key into each project once, which at least reduces the speed at which someone could get to the page.

Another option might be to use a Configure value to control whether it's available or not, but since the script is standalone and not a Pages template file, that might be difficult to import quickly/succinctly.

Or we could mark the file with @TODO: Remove me right out of the gate, but then there's no benefit in being able to count on it in every hosting environment.

On the extreme end of things, we could update the _pi.php script to limit access from loadsysdev's IP, then add a Shell Script that would: 1. SSH into loadsysdev, 2. Curl the _pi.php output into a file, 3. SCP file back to your Mac, 4. open the file in your default browser.

I'm open to further suggestions. I think there's value in having it out there, but keeping it relatively secure is something we should also take into account. My vote is for the first one, then the last one.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the standalone _pi.php script and how it is exposed in the skeleton. Compare the proposed template, Configure value, removal, and IP-restriction approaches, then establish an agreed access-control design and security behavior before implementation; the issue does not name tests or a more specific completion check.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
security
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.