Proteger el acceso directo a PDF y ZIP a menos que el usuario haya iniciado sesión (sin el complemento)

4

Trabajando en un sitio de soporte de WordPress que solo ha registrado contenido de usuario, incluidos archivos PDF y ZIP cargados.

Estoy buscando una manera de evitar el acceso directo a esos archivos PDF y ZIP dentro del directorio wp-content / uploads sin el uso de un complemento.

Leyendo las preguntas más antiguas, esto está REALMENTE cerrado (pero los comentarios están cerrados): enlace , ya que detecta directamente acceda al archivo, luego haga un inicio de sesión para guardar el archivo solicitado y verifique si el usuario ha iniciado sesión. Si no, se redirige al inicio de sesión de WordPress. Si están conectados, entonces carga el archivo.

Este es el htaccess original:

RewriteCond %{REQUEST_FILENAME} -s
RewriteRule ^wp-content/uploads/(.*)$ dl-file.php?file=$1 [QSA,L]

Sin embargo, la secuencia de comandos bloquea el acceso a todos archivos dentro del directorio wp-content / uploads, incluidas las imágenes, por lo que, a menos que el usuario haya iniciado sesión, incluso las publicaciones de blog (que no necesitan protección) ) están ocultos en la parte delantera.

Estoy tratando de revisar la RewriteRule para buscar solo archivos PDF o ZIP, de modo que solo se llame al archivo dl-file.php si se solicita un PDF o ZIP.

He intentado lo siguiente, pero devuelve "Archivo 404 no encontrado". cuando se accede a un PDF o ZIP cuando el usuario ha iniciado sesión, por lo que aunque la verificación del tipo de archivo parece estar funcionando, la verificación dl-file.php está fallando.

RewriteCond %{REQUEST_FILENAME} -s
RewriteRule ^wp-content/uploads/([^/]*\.(pdf|zip))$ filecheck.php?file=$1 [QSA,L]

¿Cómo revisaría esto para que solo llame a dl-file.php si se solicita un archivo pdf o zip, pero aún así pase la información correcta a dl-file.php?

Gracias, Jonathon

/*
 * dl-file.php
 *
 * Protect uploaded files with login.
 * 
 * @link http://wordpress.stackexchange.com/questions/37144/protect-wordpress-uploads-if-user-is-not-logged-in
 * 
 * @author hakre <http://hakre.wordpress.com/>
 * @license GPL-3.0+
 * @registry SPDX
 */

require_once('wp-load.php');

is_user_logged_in() || auth_redirect();

list($basedir) = array_values(array_intersect_key(wp_upload_dir(), array('basedir' => 1)))+array(NULL);

$file =  rtrim($basedir,'/').'/'.str_replace('..', '', isset($_GET[ 'file' ])?$_GET[ 'file' ]:'');
if (!$basedir || !is_file($file)) {
    status_header(404);
    die('404 &#8212; File not found.');
}

$mime = wp_check_filetype($file);
if( false === $mime[ 'type' ] && function_exists( 'mime_content_type' ) )
    $mime[ 'type' ] = mime_content_type( $file );

if( $mime[ 'type' ] )
    $mimetype = $mime[ 'type' ];
else
    $mimetype = 'image/' . substr( $file, strrpos( $file, '.' ) + 1 );

header( 'Content-Type: ' . $mimetype ); // always send this
if ( false === strpos( $_SERVER['SERVER_SOFTWARE'], 'Microsoft-IIS' ) )
    header( 'Content-Length: ' . filesize( $file ) );

$last_modified = gmdate( 'D, d M Y H:i:s', filemtime( $file ) );
$etag = '"' . md5( $last_modified ) . '"';
header( "Last-Modified: $last_modified GMT" );
header( 'ETag: ' . $etag );
header( 'Expires: ' . gmdate( 'D, d M Y H:i:s', time() + 100000000 ) . ' GMT' );

// Support for Conditional GET
$client_etag = isset( $_SERVER['HTTP_IF_NONE_MATCH'] ) ? stripslashes( $_SERVER['HTTP_IF_NONE_MATCH'] ) : false;

if( ! isset( $_SERVER['HTTP_IF_MODIFIED_SINCE'] ) )
    $_SERVER['HTTP_IF_MODIFIED_SINCE'] = false;

$client_last_modified = trim( $_SERVER['HTTP_IF_MODIFIED_SINCE'] );
// If string is empty, return 0. If not, attempt to parse into a timestamp
$client_modified_timestamp = $client_last_modified ? strtotime( $client_last_modified ) : 0;

// Make a timestamp for our most recent modification...
$modified_timestamp = strtotime($last_modified);

if ( ( $client_last_modified && $client_etag )
    ? ( ( $client_modified_timestamp >= $modified_timestamp) && ( $client_etag == $etag ) )
    : ( ( $client_modified_timestamp >= $modified_timestamp) || ( $client_etag == $etag ) )
    ) {
    status_header( 304 );
    exit;
}

// If we made it this far, just serve the file
readfile( $file );
    
pregunta d38 29.09.2017 - 19:54

1 respuesta

3
RewriteCond %{REQUEST_FILENAME} -s
RewriteRule ^wp-content/uploads/([^/]*\.(pdf|zip))$ filecheck.php?file=$1 [QSA,L]

¿Esto realmente se ve bien, excepto si tiene subdirectorios adicionales dentro del directorio /uploads ? Una alternativa es incluir una condición adicional en la regla original que solo reescribe la solicitud si la solicitud termina en .pdf o .zip . Por ejemplo:

RewriteCond %{REQUEST_URI} \.(pdf|zip)$ [NC]
RewriteCond %{REQUEST_FILENAME} -s
RewriteRule ^wp-content/uploads/(.*)$ dl-file.php?file=$1 [QSA,L]

Realmente no debería importar, pero asegúrese de que esto vaya antes del controlador frontal de WordPress.

    
respondido por el MrWhite 29.09.2017 - 23:26

Lea otras preguntas en las etiquetas