Browse Source

Merge pull request #10593 from nextcloud/bugfix/noid/increase-user-status-limitation

feat(user_status): Increase participant limit to 1k users
pull/10317/head
Joas Schilling 2 years ago
committed by GitHub
parent
commit
49297fa658
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 7
      lib/Config.php
  2. 2
      lib/Controller/RoomController.php

7
lib/Config.php

@ -43,6 +43,13 @@ class Config {
public const SIGNALING_TICKET_V1 = 1;
public const SIGNALING_TICKET_V2 = 2;
/**
* Currently limiting to 1k users because the user_status API would yield
* an error on Oracle otherwise. Clients should use a virtual scrolling
* mechanism so the data should not be a problem nowadays
*/
public const USER_STATUS_INTEGRATION_LIMIT = 1000;
protected array $canEnableSIP = [];
public function __construct(

2
lib/Controller/RoomController.php

@ -688,7 +688,7 @@ class RoomController extends AEnvironmentAwareController {
if ($this->userId !== null
&& $includeStatus
&& count($participants) < 100
&& count($participants) < Config::USER_STATUS_INTEGRATION_LIMIT
&& $this->appManager->isEnabledForUser('user_status')) {
$userIds = array_filter(array_map(static function (Participant $participant) {
if ($participant->getAttendee()->getActorType() === Attendee::ACTOR_USERS) {

Loading…
Cancel
Save