Use file_access() instead of file_entity_access()

Postponed until updated release of File Entity.

The patch has to be modified a little bit. The change of the access arguments is correct, but the access callback has to stay unmodified at "file_entity_access"

...set to NW as per #4

Fixed the patch

Works for me :)

Patch works for me too.

please prioritize. impeding projects require Media Colorbox (not sufficing with just Colorbox, and there's no time to rebuild Views)

#6 Works for me too, i will come to the same solution after being researching for a solution about permission

#6 Didn't work for me against media_colorbox-1.x-dev

I downloaded to media_colorbox folder and patched like so: patch -p1 < 1730730-6-file_access.patch

#6 Worked for me but I had to clear the caches.

didn't work for me

patch #6 worked fine for me. Need to clear cache first. Thx!!

I agree with #14 but you need to clear just menu cache

#6 worked for me too. Clear cache.

#5 works great! As the thread says...clearing the cache is a must.

#6 worked for me too. Clear cache.

Solved the "This image failed to load." message I was getting for YouTube Popups.

Seems like this has been reviewed and tested.

works!!!

please commit =)

Confirm fixing permission problem, please commit.

The module breaks badly without this patch, does not look good on a production site, maintainers should really commit this :)

Just echoing the chorus - the patch seems to fix this issue. Please commit!

With no disrespect intended toward maintainers, but given the 26 weeks that there has been no commit for this module, wouldn't it be useful for the community to be able to flag a module as having critical issues and have such a warning appear on the main project page to help other assess whether they want to use the module right now or not?

Again, I would like to clarify that this recommendation is in no way intended to undermine the work of the maintainers, but I feel community feedback would be a real time saver for a lot of end users.

What do you think?

good idea!

Completely agree with #24, maybe this module Maintenance status should be "Minimally mantained" instead "Actively maintained"

My total respect for module maintainers, nothing as far of my (i think our) intention is undermine the work of the maintainers.

Fixed the issue myself in exactly the same manner and THEN found this issue/patch as I was about to submit mine!
I agree with the others. The module does not work for anonymous users without this fix.
kmadel, I'm willing to volunteer as a backup maintainer if you'd like...

I've filed #1134450: Automatically degrade maintenance and development status of projects over time a couple of years ago, but no response :/

Marked #1844912: File access problem as duplicate.

Wrote the maintainer a PM, hoping for a response.

Patch works for me too. Thanks so much!

To get this committed let's get some new project maintainers or co-maintainers. Please support this issue #1952272: Media Colorbox appears unsupported with reviewed/tested patches waiting

patch in #6 fixed my 403 for users viewing youtube videos in a colorbox

The fix has been commited to the dev branch.

I'm still getting this file issue "Request unsuccessful: Forbidden" with 7.x-1.x-dev

the same error after 3 years!!! disgusting :-(

Did we really go 2+ years with only 1 person testing this? It's possible I made a mistake on the commit 2 years ago, but it surprises me that no one would have tried it.