@@ -5,18 +5,22 @@ $urls = array( 'css' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, 'mode' => 'resource', 'file' => 'css', )), 'css-ie6' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, 'mode' => 'resource', 'file' => 'css-ie6', )), 'css-ie7' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, 'mode' => 'resource', 'file' => 'css-ie7', )), 'js' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, 'mode' => 'resource', 'file' => 'js', )), core/util/webfiles/index.php @@ -50,16 +50,43 @@ $content_types = array( $replacements = array( 'css' => array( - '{IMG}' => SNAP_WI_URL_PATH.'?mode=resource&file=', - '../../../../assets/skins/sam/sprite.png' => SNAP_WI_URL_PATH.'?mode=resource&file=sam-assets.png' + '{IMG}' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => null, + )), + + '../../../../assets/skins/sam/sprite.png' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => 'sam-assets.png', + )), ), 'css-ie6' => array( - '{IMG}' => SNAP_WI_URL_PATH.'?mode=resource&file=', - '../../../../assets/skins/sam/sprite.png' => SNAP_WI_URL_PATH.'?mode=resource&file=sam-assets.png' + '{IMG}' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => null, + )), + + '../../../../assets/skins/sam/sprite.png' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => 'sam-assets.png', + )), ), 'css-ie7' => array( - '{IMG}' => SNAP_WI_URL_PATH.'?mode=resource&file=', - '../../../../assets/skins/sam/sprite.png' => SNAP_WI_URL_PATH.'?mode=resource&file=sam-assets.png' + '{IMG}' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => null, + )), + + '../../../../assets/skins/sam/sprite.png' => Snap_Request::makeURL(array( + 'key' => SNAP_WI_KEY, + 'mode' => 'resource', + 'file' => 'sam-assets.png', + )), ), ); core/util/webfiles/resource.php @@ -14,10 +14,18 @@ define('SNAP_WI_TEST_PATH', '/path/to/my/tests/'); // once more without weird script url hackery define('SNAP_WI_URL_PATH', 'http://www.example.com/path/to/snaptest_web.dist.php'); -// STEP 4: set the matching path Files matching this pattern will be testable +// STEP 4: Set a key. You only need this if you are putting your +// tests on a publicly accessible server. This also offers very minimal +// protection. If you're actually going this far in the protecting bit, +// it is probably time to read up on .htaccess and possibly write your own +// web stub to do security. +// A blank string means no key will be used. +define('SNAP_WI_KEY', ''); + +// STEP 5: set the matching path Files matching this pattern will be testable define('SNAP_WI_TEST_MATCH', '^.*\.stest\.php$'); -// STEP 5: Relax, you're done. +// STEP 6: Relax, you're done. // Go to http://www.example.com/path/to/snaptest_web.dist.php // -------------------------------------------------------------------------- snaptest_web.dist.php @@ -15,8 +15,17 @@ require_once dirname(__FILE__) . DIRECTORY_SEPARATOR . 'constants.php'; $options = Snap_Request::getLongOptions(array( 'mode' => 'index', + 'key' => '', )); +if (SNAP_WI_KEY) { + if ($options['key'] != SNAP_WI_KEY) { + echo "\n"; + echo "SnapTest Web Interface: Key Mismatch\n"; + exit; + } +} + Snap_Request::setURLBase(SNAP_WI_URL_PATH); switch ($options['mode']) { snaptest_webcore.php ebac5f4f75c7ae9660f71f19ad05fbc9b72666d4 Jakob Heuser jakob@felocity.org http://github.com/Jakobo/snaptest/commit/61c9c7c209baf8622266e618755d6c755faf9ddd 61c9c7c209baf8622266e618755d6c755faf9ddd 2008-06-09T23:05:29-07:00 2008-06-09T23:05:29-07:00 added key support for web interface Signed-off-by: Jakob Heuser <jakob@felocity.org> a966fe7ed2ca628d965c874ec143252e2f0f364b Jakob Heuser jakob@felocity.org