Issue #2878513: Forbid view/edit/delete in ContactMessageAccessControlHandler
3 open threads
Merge request reports
Activity
added 11 commits
-
1a4e334f...9b9fcd70 - 10 commits from branch
project:9.3.x
- 1e7cd23b - Issue #2878513 by varsith: Adds forbidden statement
-
1a4e334f...9b9fcd70 - 10 commits from branch
47 52 ]; 48 53 } 49 54 55 /** 56 * {@inheritdoc} 57 */ 58 protected function getExpectedUnauthorizedAccessMessage($method) { 59 if ($method === 'POST') { 60 return "Message entities are not stored."; 61 } 62 return parent::getExpectedUnauthorizedAccessMessage($method); 63 } 64 65 /** 66 * Overrinding from EntityResourceTestBase base class. changed this line in version 4 of the diff
added 1 commit
- 764e0257 - Changed 'Overriding' to 'Overrides' to satisfy CSpell.
added 1 commit
- 6f837f53 - Moved overridden methods from MessageHalJsonAnonTest class to...
63 63 $admin = $this->createUser(['uid' => 4], ['administer contact forms']); 64 64 65 65 $this->assertFalse(\Drupal::entityTypeManager()->getAccessControlHandler('contact_message')->createAccess(NULL, $no_access_user)); 66 $this->assertTrue(\Drupal::entityTypeManager()->getAccessControlHandler('contact_message')->createAccess(NULL, $access_user)); 66 // Issue 2878513 - asserting false as always rejects requests. changed this line in version 7 of the diff
225 226 $request_options[RequestOptions::BODY] = $unparseable_request_body; 227 228 // DX: 403 when unparseable request body. 229 $response = $this->request('POST', $url, $request_options); 230 $this->assertResourceErrorResponse(403, $forbiddenAccessMessage, $response); 231 232 $request_options[RequestOptions::HEADERS]['Content-Type'] = static::$mimeType; 233 234 // 403 for well-formed request. Would normally return 200 for well formed 235 // request, but request forbidden so returns 403. 236 $response = $this->request('POST', $url, $request_options); 237 $this->assertResourceErrorResponse(403, $forbiddenAccessMessage, $response); 238 239 // Would normally check for correct response, but all responses forbidden, 240 // so nothing more to do. changed this line in version 9 of the diff
Please register or sign in to reply