X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=img_auth.php;h=d7125897c5f9941e629bb361aa961067ec3888ac;hb=e26b5665b5e614d5c6733ac83835cb470e20021b;hp=f1214fed36577d49b7d3742c2276eb334db5fed3;hpb=daaa7f37a16a1c2a2127ac263d9e4999018aecc0;p=lhc%2Fweb%2Fwiklou.git diff --git a/img_auth.php b/img_auth.php index f1214fed36..d7125897c5 100644 --- a/img_auth.php +++ b/img_auth.php @@ -1,119 +1,171 @@ Foo.png) -$name = wfBaseName( $path ); -if( preg_match( '!\d+px-(.*)!i', $name, $m ) ) - $name = $m[1]; -wfDebugLog( 'img_auth', "\$name is {$name}" ); - -$title = Title::makeTitleSafe( NS_IMAGE, $name ); -if( !$title instanceof Title ) { - wfDebugLog( 'img_auth', "Unable to construct a valid Title from `{$name}`" ); - wfForbidden(); -} -$title = $title->getPrefixedText(); + // Get the requested file path (source file or thumbnail) + $matches = WebRequest::getPathInfo(); + if ( !isset( $matches['title'] ) ) { + wfForbidden( 'img-auth-accessdenied', 'img-auth-nopathinfo' ); + return; + } + $path = $matches['title']; + if ( $path && $path[0] !== '/' ) { + // Make sure $path has a leading / + $path = "/" . $path; + } -// Check the whitelist if needed -if( !$wgUser->getId() && ( !is_array( $wgWhitelistRead ) || !in_array( $title, $wgWhitelistRead ) ) ) { - wfDebugLog( 'img_auth', "Not logged in and `{$title}` not in whitelist." ); - wfForbidden(); -} + // Check for bug 28235: QUERY_STRING overriding the correct extension + $whitelist = array(); + $dotPos = strrpos( $path, '.' ); + if ( $dotPos !== false ) { + $whitelist[] = substr( $path, $dotPos + 1 ); + } + if ( !$wgRequest->checkUrlExtension( $whitelist ) ) { + return; + } -if( !file_exists( $filename ) ) { - wfDebugLog( 'img_auth', "`{$filename}` does not exist" ); - wfForbidden(); -} -if( is_dir( $filename ) ) { - wfDebugLog( 'img_auth', "`{$filename}` is a directory" ); - wfForbidden(); -} + // Get the local file repository + $repo = RepoGroup::singleton()->getRepo( 'local' ); -// Stream the requested file -wfDebugLog( 'img_auth', "Streaming `{$filename}`" ); -wfStreamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); -wfLogProfilingData(); + // Get the full file storage path and extract the source file name. + // (e.g. 120px-Foo.png => Foo.png or page2-120px-Foo.png => Foo.png). + // This only applies to thumbnails, and all thumbnails should + // be under a folder that has the source file name. + if ( strpos( $path, '/thumb/' ) === 0 ) { + $name = wfBaseName( dirname( $path ) ); // file is a thumbnail + $filename = $repo->getZonePath( 'thumb' ) . substr( $path, 6 ); // strip "/thumb" + } else { + $name = wfBaseName( $path ); // file is a source file + $filename = $repo->getZonePath( 'public' ) . $path; + } -/** - * Issue a standard HTTP 403 Forbidden header and a basic - * error message, then end the script - */ -function wfForbidden() { - header( 'HTTP/1.0 403 Forbidden' ); - header( 'Vary: Cookie' ); - header( 'Content-Type: text/html; charset=utf-8' ); - echo << - -

Access Denied

-

You need to log in to access files on this server.

- - -ENDS; - wfLogProfilingData(); - exit(); + // Check to see if the file exists + if ( !$repo->fileExists( $filename ) ) { + wfForbidden( 'img-auth-accessdenied','img-auth-nofile', $filename ); + return; + } + + $title = Title::makeTitleSafe( NS_FILE, $name ); + if ( !$title instanceof Title ) { // files have valid titles + wfForbidden( 'img-auth-accessdenied', 'img-auth-badtitle', $name ); + return; + } + + // Run hook for extension authorization plugins + if ( !wfRunHooks( 'ImgAuthBeforeStream', array( &$title, &$path, &$name, &$result ) ) ) { + wfForbidden( $result[0], $result[1], array_slice( $result, 2 ) ); + return; + } + + // Check user authorization for this title + // Checks Whitelist too + if ( !$title->userCan( 'read' ) ) { + wfForbidden( 'img-auth-accessdenied', 'img-auth-noread', $name ); + return; + } + + // Stream the requested file + wfDebugLog( 'img_auth', "Streaming `".$filename."`." ); + $repo->streamFile( $filename, array( 'Cache-Control: private', 'Vary: Cookie' ) ); } /** - * Show a 403 error for use when the wiki is public + * Issue a standard HTTP 403 Forbidden header ($msg1-a message index, not a message) and an + * error message ($msg2, also a message index), (both required) then end the script + * subsequent arguments to $msg2 will be passed as parameters only for replacing in $msg2 + * @param $msg1 + * @param $msg2 */ -function wfPublicError() { +function wfForbidden( $msg1, $msg2 ) { + global $wgImgAuthDetails; + + $args = func_get_args(); + array_shift( $args ); + array_shift( $args ); + + $msgHdr = htmlspecialchars( wfMsg( $msg1 ) ); + $detailMsgKey = $wgImgAuthDetails ? $msg2 : 'badaccess-group0'; + $detailMsg = htmlspecialchars( wfMsg( $detailMsgKey, $args ) ); + + wfDebugLog( 'img_auth', + "wfForbidden Hdr:" . wfMsgExt( $msg1, array( 'language' => 'en' ) ). " Msg: ". + wfMsgExt( $msg2, array( 'language' => 'en' ), $args ) + ); + header( 'HTTP/1.0 403 Forbidden' ); + header( 'Cache-Control: no-cache' ); header( 'Content-Type: text/html; charset=utf-8' ); echo << -

Access Denied

-

The function of img_auth.php is to output files from a private wiki. This wiki -is configured as a public wiki. For optimal security, img_auth.php is disabled in -this case. -

+

$msgHdr

+

$detailMsg

ENDS; - wfLogProfilingData(); - exit; } -