Sentry Issue: ENGINE-D

Exception: user_guid be be sent
  File "/Core/Notification/UpdateMarkers/Repository.php", line 34, in Minds\Core\Notification\UpdateMarkers\Repository::getList
    throw new \Exception('user_guid be be sent');
  File "/Core/Notification/UpdateMarkers/Manager.php", line 28, in Minds\Core\Notification\UpdateMarkers\Manager::getList
    return $this->repository->getList($opts);
  File "/Core/Groups/Membership.php", line 649, in Minds\Core\Groups\Membership::getGroupsByMember
    'limit' => 1000,
  File "/Controllers/api/v1/groups.php", line 52, in Minds\Controllers\api\v1\groups::get
    'offset' => (int) $_GET['offset'],
  File "/Api/Factory.php", line 70, in Minds\Api\Factory::build
    return $handler->$method($pages);
...
(3 additional frame(s) were not displayed)

We need to check to make sure that we check for an authenticated user before we request update markers

added scoped labels

changed weight to 2

added 1 deleted label

added scoped labels

assigned to @benhayward.ben

changed time estimate to 1h

added scoped label

added 1 deleted label and removed 1 deleted label

changed weight to 1

added scoped label and automatically removed label

mentioned in merge request engine!372 (closed)

added scoped label and automatically removed label

Have spent an hour on this but the solution I'm not happy with. Will discuss tomorrow.

added 1h of time spent at 2019-10-16

removed label

added scoped label

added scoped label and automatically removed label

added scoped label and automatically removed label

added scoped label and automatically removed label

removed 1 deleted label

added 1 deleted label

removed 1 deleted label

added 1 deleted label

removed label

This is going to take some lower level changes than just checking for the user at the start of the failing endpoint.

changed time estimate to 3h

Will discuss through the week.

added 1 deleted label and removed 1 deleted label

unassigned @benhayward.ben

added 1 deleted label and removed 1 deleted label

assigned to @benhayward.ben

changed weight to 3

added scoped label

added scoped label

removed 1 deleted label

removed label

changed milestone to %Sprint::01/01 - Xerothermic Xenartha

added scoped label and automatically removed label

added scoped label

removed label

added scoped label

This is going to require a discussion with @brianhatchet and @edgebal because we can add a check to the Factory, but I believe this is something that has existed previously and was removed.

added scoped label and automatically removed label

added scoped label and automatically removed label

changed the description

added scoped label and automatically removed label

added scoped label and automatically removed label

@benhayward.ben I updated the case with the appropriate fix

added scoped label and automatically removed label

changed milestone to %Sprint::01/01 - Xerothermic Xenartha

changed weight to 3

moved from engine#871 (moved)

mentioned in merge request !777

added 15m of time spent at 2020-02-12

Made an MR for that, assuming you added in the line

We need to check to make sure that we check for an authenticated user before we request update markers

added scoped label and automatically removed label

added scoped label and automatically removed label

added scoped label and automatically removed label

added scoped label and automatically removed label

added scoped label and automatically removed label

Sockets don't appear to be working on sandbox, nor does notifications.

added scoped label and automatically removed label

socket jwt in the review site's settings is changeme - I suspect this is not matching on the sockets server

added 20m of time spent at 2020-02-12

added scoped label and automatically removed label

Work with @Emi to set the socket tokens on the review sites.