Problem/Motivation

WebTestBase::drupalGetToken() is never used. Additionally #2245003: Use a random seed instead of the session_id for CSRF token generation changed the way CSRF tokens are generated but this change was never reflected in WebTestBase::drupalGetToken().

Proposed resolution

Remove it.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 2328995-remove-web-test-base-drupal-get-token.patch699 bytesznerol

Comments

znerol’s picture

znerol commented
Status: Active » Needs review
StatusFileSize
new 2328995-remove-web-test-base-drupal-get-token.patch699 bytes
andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

No usage is left, +1 to remove

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Nice catch.

Committed and pushed to 8.x. Thanks!

  • webchick committed 68cd18c on 8.0.x 
    Issue #2328995 by znerol: Remove unused and defective WebTestBase::...

Status: Fixed » Closed (fixed)

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