[stable28] fix(caldav): Do not load IMipPlugin before user auth and session is c… #47791
Conversation
SebastianKrupinski
requested review from
ChristophWurst,
miaulalala and
tcitworld
as code owners
miaulalala
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| $senderName = $this->userManager->getDisplayName($this->userId); | ||
| // Due to a bug in sabre, the senderName property for an iTIP message can actually also be a VObject Property | ||
| // If the iTIP message senderName is null or empty use the user session name as the senderName | ||
| if (($iTipMessage->senderName instanceof Parameter) && !empty(trim($iTipMessage->senderName->getValue()))) { |
Check notice
Code scanning / Psalm
PossiblyNullArgument Note
| // Due to a bug in sabre, the senderName property for an iTIP message can actually also be a VObject Property | ||
| // If the iTIP message senderName is null or empty use the user session name as the senderName | ||
| if (($iTipMessage->senderName instanceof Parameter) && !empty(trim($iTipMessage->senderName->getValue()))) { | ||
| $senderName = trim($iTipMessage->senderName->getValue()); |
Check notice
Code scanning / Psalm
PossiblyNullArgument Note
| } elseif (is_string($iTipMessage->senderName) && !empty(trim($iTipMessage->senderName))) { | ||
| $senderName = trim($iTipMessage->senderName); | ||
| } elseif ($this->userSession->getUser() !== null) { | ||
| $senderName = trim($this->userSession->getUser()->getDisplayName()); |
Check notice
Code scanning / Psalm
PossiblyNullReference Note
| @@ -179,12 +180,10 @@ | |||
|
|
|||
| // calendar plugins | |||
| if ($this->requestIsForSubtree(['calendars', 'public-calendars', 'system-calendars', 'principals'])) { | |||
| $this->server->addPlugin(new DAV\Sharing\Plugin($authBackend, \OC::$server->getRequest(), \OC::$server->getConfig())); | |||
Check notice
Code scanning / Psalm
DeprecatedMethod Note
| @@ -179,12 +180,10 @@ | |||
|
|
|||
| // calendar plugins | |||
| if ($this->requestIsForSubtree(['calendars', 'public-calendars', 'system-calendars', 'principals'])) { | |||
| $this->server->addPlugin(new DAV\Sharing\Plugin($authBackend, \OC::$server->getRequest(), \OC::$server->getConfig())); | |||
Check notice
Code scanning / Psalm
DeprecatedMethod Note
| @@ -312,6 +310,18 @@ | |||
| \OC::$server->getCommentsManager(), | |||
| $userSession | |||
| )); | |||
| if (\OC::$server->getConfig()->getAppValue('dav', 'sendInvitations', 'yes') === 'yes') { | |||
Check notice
Code scanning / Psalm
DeprecatedMethod Note
…reated Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
solracsf
force-pushed
the
backport/45098/stable28
branch
from
7c569f1
to
2ac9a1b
Compare
Back port for #45081
Do not load IMipPlugin before user is authenticated and user session is initialized