Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

gabesullice’s picture

Assigned: Unassigned » gabesullice
Wim Leers’s picture

Assigned: gabesullice » Wim Leers

Gabe reproduced it, then he also got stuck on figuring out why. Did some debugging, found the root cause.

#2765959: Make 4xx REST responses cacheable by (Dynamic) Page Cache + comprehensive cacheability test coverage introduced a fix to UserAccessControlHandler, which causes those without view access to get an access result that contains the cache tag of the User entity you tried to view.

Proposed fix upcoming.

Wim Leers’s picture

Assigned: Wim Leers » Unassigned
Status: Active » Needs review
FileSize
1.39 KB
Wim Leers’s picture

Status: Needs review » Reviewed & tested by the community

Green! :)

Let's get this in, so that daily test runs are green again, and so that #2929935: Remove JSON API's ScalarNormalizer: it's no longer necessary thanks to #2751325's PrimitiveDataNormalizer is unblocked.

  • Wim Leers committed 1c49c7f on 8.x-1.x
    Issue #2931844 by Wim Leers: Make JsonApiDocumentTopLevelNormalizerTest...
Wim Leers’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.