PrivateTempStore throws exception if session is not started (drush users)

Yes that fixes things, thanks @heddn

Yes, works perfectly, thanks!

With migrate 8.x-4.0-beta3, patch won't apply. So I'll try the dev branch.

patch -p1 < 2976862.patch
patching file migrate_tools.module
Hunk #1 FAILED at 36.
1 out of 1 hunk FAILED -- saving rejects to file migrate_tools.module.rej
dhaley-30197s:migrate_tools dhaley$ cat migrate_tools.module.rej
***************
*** 36,41 ****
/** @var \Drupal\Core\TempStore\PrivateTempStoreFactory $store */
$tempStoreFactory = \Drupal::service('tempstore.private');
$store = $tempStoreFactory->get('migrate_tools');
// Get the list of changed migrations.
$migrationsChanged = $store->get('migrations_changed');
if (isset($store) && (is_array($migrationsChanged))) {
--- 36,48 ----
/** @var \Drupal\Core\TempStore\PrivateTempStoreFactory $store */
$tempStoreFactory = \Drupal::service('tempstore.private');
$store = $tempStoreFactory->get('migrate_tools');
+ // TODO: remove work-around after
+ // https://www.drupal.org/project/drupal/issues/2860341 is fixed.
+ if (!\Drupal::request()->hasSession()) {
+ $session = \Drupal::service('session');
+ \Drupal::request()->setSession($session);
+ $session->start();
+ }
// Get the list of changed migrations.
$migrationsChanged = $store->get('migrations_changed');
if (isset($store) && (is_array($migrationsChanged))) {

Exposed core bug #2860341: PrivateTempStore->getOwner Attempts to access possibly unset Request Session is fixed and committed. And so will be in D8.7 and the next 8.6.x release.
Is it best to undo this patch in this thread (and if so, when?)
Or should it be done in a followup issue?

## EDIT
Trying to change the status I just answered my own question I guess. Created a new follow up issue: #3001798: Core patch is committed, follow up to remove workaround