Kein ORGANIZER / ATTENDEE im ics

[jgruenewald]

Hallo Forum,

beim Einbinden des Gruppen-Kalenders über groupdav.php im Thunderbird-Lightning (hat mir dem eigentlichen Problem aber nichts zu tun) ist mir aufgefallen, dass bei Terminen, die nur 1 Teilnehmer haben weder “ATTENDEE”, noch “ORGANIZER” im ics gesetzt sind. Das entspricht auch dem programmierten Verhalten:

// class.calendar_ical.inc.php
Zeile 451:
// do not include event owner/ORGANIZER as participant in his own calendar, if he is only participant
if (count($event['participants']) == 1 && $event['owner'] == $uid) continue;

Zeile 580:
// do NOT use ORGANIZER for events without further participants or a different organizer
if (count($event['participants']) > 1 || !isset($event['participants'][$event['owner']])) {
    // ORGANIZER would be added here
}

Hat den unschönen Nebeneffekt, dass wenn ein Mitarbeiter, der z.B. seinen Urlaub einträgt und damit gleichzeitig Owner und einziger Participant des Termins ist, leider nicht als solcher für mich im Lightning erkennbar ist. Ist hier eine Konfigurationsmöglichkeit bekannt, die mich das ändern lässt?

Gruß
Jan

[RalfBecker]

Hi Jan,

ich würde mal sagen das sollte generell nicht so sein, sprich dafür macht eine Konfig keinen Sinn.

Versuch mal die folgende Änderung:

diff --git a/calendar/inc/class.calendar_ical.inc.php b/calendar/inc/class.calendar_ical.inc.php
index 640c726167..0a88e4e379 100644
--- a/calendar/inc/class.calendar_ical.inc.php
+++ b/calendar/inc/class.calendar_ical.inc.php
@@ -448,7 +448,7 @@ class calendar_ical extends calendar_boupdate
                                                        $quantity = $role = null;
                                                        calendar_so::split_status($status, $quantity, $role);
                                                        // do not include event owner/ORGANIZER as participant in his own calendar, if he is only participant
-                                                       if (count($event['participants']) == 1 && $event['owner'] == $uid) continue;
+                                                       if (count($event['participants']) == 1 && $event['owner'] == $uid && $uid == $this->user) continue;

                                                        if (!($info = $this->resource_info($uid))) continue;

@@ -578,7 +578,7 @@ class calendar_ical extends calendar_boupdate
                                                $organizerUID = $event['owner'];
                                        }
                                        // do NOT use ORGANIZER for events without further participants or a different organizer
-                                       if (count($event['participants']) > 1 || !isset($event['participants'][$event['owner']]))
+                                               if (count($event['participants']) > 1 || !isset($event['participants'][$event['owner']]) || $event['owner'] != $this->user)
                                        {
                                                $attributes['ORGANIZER'] = $organizerURL;
                                                $parameters['ORGANIZER']['CN'] = $organizerCN;

Wie man den Code im Container patched, habe ich hier schon ein paar mal geschrieben.

Ralf

[jgruenewald]

Hi Ralf,

Code eingebaut (eine Anleitung habe ich auf die Schnelle im Forum nicht gefunden, aber ich vermute da steht dann sowas wie: docker exec -ti egroupware /bin/bash -> nano file.php -> service php7.3-fpm reload; heißt natürlich auch, dass die Änderungen beim nächsten Neustart wieder weg sind).

Grundsätzlich: funktioniert. Vielen Dank. Aber: warum gibt es denn Ausschlus des Attendees und Organizers überhaupt? Schadet es, wenn ich bei einem Termin, der mir gehört auch sehe, dass er mir gehört?

Wichtige Anmerkung; aber OFF-TOPIC
Diese Warning wird bei uns am laufenden Band produziert:
PHP Warning: session_id(): Cannot change session id when session is active in /usr/share/egroupware/api/src/Session.php on line 547
Extrem unglücklich ist dabei der Hinweis auf /api/src/Header/Authenticate.php(109) im Stacktrace, denn bei dem Function-Call steckt ein Passwort im Klartext und das taucht dann leider auch genau so im Log auf.

Jan

[RalfBecker]

Code eingebaut (eine Anleitung habe ich auf die Schnelle im Forum nicht gefunden, aber ich vermute da steht dann sowas wie: docker exec -ti egroupware /bin/bash -> nano file.php -> service php7.3-fpm reload; heißt natürlich auch, dass die Änderungen beim nächsten Neustart wieder weg sind).

Wenn Du die Änderung in /usr/share/egroupware-sources einbaust, überstehen sie einen Neustart, aber keine neue Version.

Grundsätzlich: funktioniert.

Danke, dann werde ich das so commiten.

warum gibt es denn Ausschlus des Attendees und Organizers überhaupt? Schadet es, wenn ich bei einem Termin, der mir gehört auch sehe, dass er mir gehört?

Weil es damit diverse Probleme gab und andere Server in der Situation (eigener Termin im eigenen Kalender) das auch nicht machen.

Diese Warning wird bei uns am laufenden Band produziert:

PHP Warning: session_id(): Cannot change session id when session is active in /usr/share/egroupware/api/src/Session.php on line 547

Extrem unglücklich ist dabei der Hinweis auf /api/src/Header/Authenticate.php(109) im Stacktrace, denn bei dem Function-Call steckt ein Passwort im Klartext und das taucht dann leider auch genau so im Log auf.

Muss ich mir anschauen.

Ralf

[RalfBecker]

Muss ich mir anschauen.

Damit sollte das behoben sein (Zeilennummern sind ein paar Zeilen verschoben, weil es inzwischen noch einen Commit in der Datei gab):

diff --git a/api/src/Session.php b/api/src/Session.php
index 3cabed08ed..c800446693 100644
--- a/api/src/Session.php
+++ b/api/src/Session.php
@@ -550,7 +550,10 @@ class Session
                        // --> allows this stateless protocolls which use basic auth to use sessions!
                        if (($this->sessionid = self::get_sessionid(true)))
                        {
-                               session_id($this->sessionid);
+                               if (session_status() !== PHP_SESSION_ACTIVE)    // gives warning including password
+                               {
+                                       session_id($this->sessionid);
+                               }
                        }
                        else
                        {

Ralf

[jgruenewald]

Warning mit Klartext-Passwort wird nicht mehr angezeigt und groupdav funktioniert weiterhin sowohl über Lightning, als auch über den Browser.

Vielen Dank für die zügige und zielgerichtete Unterstützung.

[RalfBecker]

Vielen Dank für die zügige und zielgerichtete Unterstützung.

Gerne und danke für das Testen.

Beider Änderungen sind commited und werden mit dem nächsten Maintenance Release kommen.

Ralf