Problem/Motivation
As an editor on drupal.nl i expect to be able to upload files, this was possible before last version.
Steps to reproduce
Deploy most recent code for https://druplal.nl and upload a file with an existing database.
Notice that the version of the issue might not be the actual code of what is deployed.
Proposed resolution
Determine if this is devops or code, and fix it.
Remaining tasks
- fix the code
- redeploy.
User interface changes
none
API changes
none
Data model changes
none
Issue fork la_eu-3587540
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3587540-file-upload-broken
changes, plain diff MR !90
Comments
Comment #2
taran2lhi @florisg what exactly does not work ?
Comment #3
florisg commentedFile upload in production site: https://drupal.nl
For example when uploading an image.
This is something to investigate, not postpond.
Comment #4
evilargest commentedHey @florisg, could you provide STR for this problem?
I see that there is a similar problem with the WebP derivative image generation, so images on some pages can be broken (see https://www.drupal.nl/partners/ronder), which might be related. However, the issue is fixed, and it does not reproduce anymore.
Trying to upload an image to a media entity worked out fine without any crits/problems on local/staging environments. Is there anything else we should pay attention to?
Comment #5
taran2lComment #6
taran2lThe source of the issue is probably:Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException: Non-reusable blocks must set an access dependency for access control. in Drupal\Core\Routing\AccessAwareRouter->checkAccess()caused by #3047022: Layout builder fails to assign inline block access dependencies for the overrides section storage on entities with pending revisionsnope
Comment #8
evilargest commentedComment #9
taran2lComment #10
taran2lComment #11
taran2lI think this issue was drupal.nl specific due to the issue with webp + tinypng modules combination, and it's already fixed on production
Comment #13
taran2lThe open MR should be moved to #3588882: Media Upload and Media selection does not work