Prevent direct access to a php include file

后端 未结 30 967
盖世英雄少女心
盖世英雄少女心 2020-11-22 06:32

I have a php file which I will be using as exclusively as an include. Therefore I would like to throw an error instead of executing it when it\'s accessed directly by typing

30条回答
  •  旧巷少年郎
    2020-11-22 07:01

    My answer is somewhat different in approach but includes many of the answers provided here. I would recommend a multipronged approach:

    1. .htaccess and Apache restrictions for sure
    2. defined('_SOMECONSTANT') or die('Hackers! Be gone!');

    HOWEVER the defined or die approach has a number of failings. Firstly, it is a real pain in the assumptions to test and debug with. Secondly, it involves horrifyingly, mind-numbingly boring refactoring if you change your mind. "Find and replace!" you say. Yes, but how sure are you that it is written exactly the same everywhere, hmmm? Now multiply that with thousands of files... o.O

    And then there's .htaccess. What happens if your code is distributed onto sites where the administrator is not so scrupulous? If you rely only on .htaccess to secure your files you're also going to need a) a backup, b) a box of tissues to dry your tears, c) a fire extinguisher to put out the flames in all the hatemail from people using your code.

    So I know the question asks for the "easiest", but I think what this calls for is more "defensive coding".

    What I suggest is:

    1. Before any of your scripts require('ifyoulieyougonnadie.php'); (not include() and as a replacement for defined or die)
    2. In ifyoulieyougonnadie.php, do some logic stuff - check for different constants, calling script, localhost testing and such - and then implement your die(), throw new Exception, 403, etc.

      I am creating my own framework with two possible entry points - the main index.php (Joomla framework) and ajaxrouter.php (my framework) - so depending on the point of entry, I check for different things. If the request to ifyoulieyougonnadie.php doesn't come from one of those two files, I know shenanigans are being undertaken!

      But what if I add a new entry point? No worries. I just change ifyoulieyougonnadie.php and I'm sorted, plus no 'find and replace'. Hooray!

      What if I decided to move some of my scripts to do a different framework that doesn't have the same constants defined()? ... Hooray! ^_^

    I found this strategy makes development a lot more fun and a lot less:

    /**
     * Hmmm... why is my netbeans debugger only showing a blank white page 
     * for this script (that is being tested outside the framework)?
     * Later... I just don't understand why my code is not working...
     * Much later... There are no error messages or anything! 
     * Why is it not working!?!
     * I HATE PHP!!!
     * 
     * Scroll back to the top of my 100s of lines of code...
     * U_U
     *
     * Sorry PHP. I didn't mean what I said. I was just upset.
     */
    
     // defined('_JEXEC') or die();
    
     class perfectlyWorkingCode {}
    
     perfectlyWorkingCode::nowDoingStuffBecauseIRememberedToCommentOutTheDie();
    

提交回复
热议问题