WPIntell

Source evidence

Running on WordPress v4.9.1 and PHP v7

Media Vault · support · 2018-01-09T23:58:00+00:00

complaintsentiment
highseverity
1.0relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

6 / 32 rows with source links

18.8% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

26 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Kondor with a K unresolved
One of my WordPress clients relies heavily on the Media Vault plugin for their membership website. In the years since installing this plugin, the support has died off and it has received the dreaded “This plugin hasn’t been updated in over 2 years” message. It’s still a great plugin, and having looked over a good portion of the code, I think it’s very-well written. Whether the original developer didn’t have enough time or didn’t think enough people used their plugin, I still think Media Vault is worth maintaining and still using. Hopefully the community can restart Media Vault, much like they did with Postman SMTP becoming Post SMTP. While using this plugin on PDFs and Word document files, I received the following error message. The wp_get_attachment_metadata was not returning the file URL or any information for that matter. PHP Warning: Illegal string offset ‘sizes’ To fix, I incorporated the following code to the /wp-content/plugins/media-vault/includes/mgjp-functions.php file: COMMENT OUT THIS LINE: // $meta = wp_get_attachment_metadata( $attachment_id ); USE THIS FUNCTION INSTEAD: $meta = array(); $meta[‘file’] = get_attached_file($attachment_id); $meta[‘sizes’] = ”; Several topics have also mentioned that files can still be accessed with this plugin. To resolve this, I wrote a custom download script that checks the permissions of the page the file was uploaded to in order to double check the logged-in user has enough permission to retrieve the file. To redirect to that download script, I have added this in the .htaccess file: # Don’t redirect images RewriteRule ^wp-content/uploads/(.*)\.jpg$ – [R=301,L] RewriteRule ^wp-content/uploads/(.*)\.png$ – [R=301,L] RewriteRule ^wp-content/uploads/(.*)\.gif$ – [R=301,L] # Redirect to check for permissions RewriteRule ^wp-content/uploads/(.*)$ /wp-content/themes/mytheme/download\.php?file=$1 [R=301,L,QSA] In the future, I may also post the download script in the hope that it encourages the original developer to pickup the project or perhaps pass it onto somebody who will. This would be great, as this my sites rely heavily on this plugin, and continued support will be very appreciated. I forked this on Github and resolved a vulnerability in mv-file-handler.php which allowed authenticated users to access other users files. If you would like to send your changes I can add to the repo. https://github.com/derricksmith/wordpress-plugin-media-vault

Comments

2 shown
spacetimepartners 2018-02-20T14:50:00+00:00

This would be great, as this my sites rely heavily on this plugin, and continued support will be very appreciated.

derricksmith01 2018-03-09T22:20:00+00:00

I forked this on Github and resolved a vulnerability in mv-file-handler.php which allowed authenticated users to access other users files. If you would like to send your changes I can add to the repo. https://github.com/derricksmith/wordpress-plugin-media-vault