Problem/Motivation

Automated sync throws errors when interfacing with the core crypt class.

Steps to reproduce

run cron or periodically check the dblog

Deprecated function: hash(): Passing null to parameter #2 ($data) of type string is deprecated in Drupal\Component\Utility\Crypt::hashBase64() (line 49 of /srv/drupal/website/dev/code/project/docroot/core/lib/Drupal/Component/Utility/Crypt.php)
#0 /srv/drupal/website/dev/code/project/docroot/core/includes/bootstrap.inc(164): _drupal_error_handler_real()
#1 [internal function]: _drupal_error_handler()
#2 /srv/drupal/website/dev/code/project/docroot/core/lib/Drupal/Component/Utility/Crypt.php(49): hash()
#3 /srv/drupal/website/dev/code/project/docroot/modules/contrib/acquia_connector/src/EventSubscriber/KernelTerminate/AcquiaTelemetry.php(266): Drupal\Component\Utility\Crypt::hashBase64()
#4 /srv/drupal/website/dev/code/project/docroot/modules/contrib/acquia_connector/src/EventSubscriber/KernelTerminate/AcquiaTelemetry.php(254): Drupal\acquia_connector\EventSubscriber\KernelTerminate\AcquiaTelemetry->getUserId()
#5 /srv/drupal/website/dev/code/project/docroot/modules/contrib/acquia_connector/src/EventSubscriber/KernelTerminate/AcquiaTelemetry.php(169): Drupal\acquia_connector\EventSubscriber\KernelTerminate\AcquiaTelemetry->createEvent()
#6 /srv/drupal/website/dev/code/project/docroot/modules/contrib/acquia_connector/src/EventSubscriber/KernelTerminate/AcquiaTelemetry.php(110): Drupal\acquia_connector\EventSubscriber\KernelTerminate\AcquiaTelemetry->sendTelemetry()
#7 [internal function]: Drupal\acquia_connector\EventSubscriber\KernelTerminate\AcquiaTelemetry->onTerminateResponse()
#8 /srv/drupal/website/dev/code/project/docroot/core/lib/Drupal/Component/EventDispatcher/ContainerAwareEventDispatcher.php(111): call_user_func()
#9 /srv/drupal/website/dev/code/project/vendor/symfony/http-kernel/HttpKernel.php(115): Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher->dispatch()
#10 /srv/drupal/website/dev/code/project/docroot/core/lib/Drupal/Core/StackMiddleware/StackedHttpKernel.php(63): Symfony\Component\HttpKernel\HttpKernel->terminate()
#11 /srv/drupal/website/dev/code/project/docroot/core/lib/Drupal/Core/DrupalKernel.php(688): Drupal\Core\StackMiddleware\StackedHttpKernel->terminate()
#12 /srv/drupal/website/dev/code/project/docroot/index.php(22): Drupal\Core\DrupalKernel->terminate()
#13 {main}
.

Error adds entries to the logs on each cron run.

Drupal 10.1.6
PHP: 8.1
Apache: 2.4.52
MariaDB: 10.6.12

Acquia Connector: 4.0.5

Issue fork acquia_connector-3398737

Command icon 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:

About issue forks

Comments

emptyvoid created an issue. See original summary.

japerry made their first commit to this issue’s fork.

japerry changed the visibility of the branch 4.x to hidden.

japerry opened merge request !45

japerry’s picture

japerry
Location KOMK
commented
Status: Active » Needs review

Need to validate this with ACMS team, but this PR will just make log requests anonymous if the site uuid cannot be found.

  • japerry committed 93cf8236 on 4.x 
    Issue #3398737 by japerry, emptyvoid: Crypt parameter validation missing
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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