Calendar GroupDAV Crash

derSascha 2018-12-16 16:50:01 UTC #1

Hi,

ich versuche gerade Termine mit DAVdroid zu übertragen. Leider stürzt die E-Groupware dabei immer folgendem Fehler ab:

PHP Warning: array_intersect(): Argument #2 is not an array in /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php on line 818
#1 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(818): array_intersect(Array, NULL)
#2 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(800): calendar_groupdav::isParticipant(Array, -6)
#3 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(691): calendar_groupdav::get_series('calendar-35-53d...', Object(calendar_boupdate), false, -6)
#4 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(391): calendar_groupdav->iCal(Array, -6, NULL, false)
#5 /usr/share/egroupware/api/src/CalDAV/PropfindIterator.php(149): calendar_groupdav->propfind_callback('/Bundesbuero/ca...', Array, Array)
#6 /usr/share/egroupware/api/src/CalDAV/PropfindIterator.php(172): EGroupware\\Api\\CalDAV\\PropfindIterator->next()
#7 /usr/share/egroupware/api/src/WebDAV/Server.php(794): EGroupware\\Api\\CalDAV\\PropfindIterator->rewind()
#8 /usr/share/egroupware/api/src/WebDAV/Server.php(757): HTTP_WebDAV_Server->multistatus_responses(Object(EGroupware\\Api\\CalDAV\\PropfindIterator), Array)
#9 /usr/share/egroupware/api/src/CalDAV.php(976): HTTP_WebDAV_Server->http_PROPFIND('REPORT')
#10 /usr/share/egroupware/api/src/WebDAV/Server.php(306): EGroupware\\Api\\CalDAV->http_REPORT()
#11 /usr/share/egroupware/api/src/CalDAV.php(2058): HTTP_WebDAV_Server->ServeRequest(NULL)
#12 /usr/share/egroupware/groupdav.php(52): EGroupware\\Api\\CalDAV->ServeRequest()
#13 {main}

Ich habe dann die Zeile hier aktiviert:

github.com

EGroupware/egroupware/blob/a361d05767e725e55d323622144f7d47d9e8e195/api/src/Accounts.php#L884


	function memberships($account_id, $just_id=false)
	{
		if (!is_int($account_id) && !is_numeric($account_id))
		{
			$account_id = $this->name2id($account_id,'account_lid','u');
		}
		if ($account_id && ($data = self::cache_read($account_id)))
		{
			$ret = $just_id && $data['memberships'] ? array_keys($data['memberships']) : $data['memberships'];
		}
		//error_log(__METHOD__."($account_id, $just_id) data=".array2string($data)." returning ".array2string($ret));
		return $ret;
	}


	/**
	 * Sets the memberships of a given account
	 *
	 * @param array $groups array with gidnumbers
	 * @param int $account_id uidnumber
	 * @return boolean true: membership changed, false: no change necessary
	 */

Es wird Accounts::memberships(-6, 1) aufgerufen, wobei -6 für eine Gruppe steht, diese keine memberships hat und damit NULL zurück gegeben wird.

Hat jemand eine Idee was man da machen kann?

Es handelt sich um die offiziellen Pakete für Ubuntu 16.04 mit Version 17.1.20181205.

Danke + Grüße
Sascha

RalfBecker 2018-12-28 08:04:10 UTC #2

Hi Sascha,

das ist eine Warning und sollte keine Auswirkung haben.

Sprich Du hast vermutlich ein anders Problem.

Schalte mal unter Einstellungen >> CalDAC/CardDAV Sync >> Logging und Fehlersuche >> Logging einschalten auf “Anfragen und komplette Antworten in das Dateiverzeichnis” und speichere.

Dann richte das Konto auf DAVdroid neu ein und gehe erneut auf Logging und Fehlersuche: unter Zeige Log des folgenden Geräts kannst Du jetzt den DAVdroid auswählen und du siehst was zwischen EGroupware und dem Client ausgetauscht wird.

Falls es da ein Problem gibt, kannst Du das Log hier hochladen.

Ralf

derSascha 2019-01-19 16:35:55 UTC #3

Hi Ralf,

Danke für die Antwort und den Tipp. Ich habe jetzt noch mal lange rumprobiert und kann jetzt folgendes Problem reproduzieren. Das ganze ist nicht exakt die gleiche Stelle aus dem ersten Beitrag, klingt aber sehr ähnlich.

Ein Benutzer legt einen Termin mit Wiederholungstyp jährlich an
Der Termin wird via DAV über einen Gruppen Kallender abgerufen (also z.B. https…/groupdav.php/some-group/calendar/13.ics)
E-Groupware liefert jetzt die Warnung zu array_intersec (siehe stacktrace) und einen leere ICS Datei womit DAVdroid wiederum nicht zurechtkommt.

DAVdroid Fehlermeldung: Received multi-get response without address data (code)

DAV debug (calendar-data ist leer):

<D:response xmlns:ns0="urn:uuid:c2f41010-65b3-11d1-a29f-00aa00c14882/" xmlns:ns2="urn:ietf:params:xml:ns:caldav">
    <D:href>/egroupware/groupdav.php/some-group/calendar/13.ics</D:href>
    <D:propstat>
        <D:prop>
            <D:getcontenttype>text/calendar; charset=utf-8; component=VEVENT</D:getcontenttype>
            <D:getetag>"13:3:1536564930"</D:getetag>
            <ns2:calendar-data/>
        </D:prop>
        <D:status>HTTP/1.1 200 OK</D:status>
    </D:propstat>
</D:response>

Warning in der E-Groupware/Apache log:

PHP Warning: array_intersect(): Argument #2 is not an array in /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php on line 818, referer: https://.../egroupware/groupdav.php/some-group/calendar/
#1 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(818): array_intersect(Array, NULL), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#2 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(800): calendar_groupdav::isParticipant(Array, -6), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#3 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(691): calendar_groupdav::get_series('calendar-13-53d...', Object(calendar_boupdate), false, -6), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#4 /usr/share/egroupware/calendar/inc/class.calendar_groupdav.inc.php(644): calendar_groupdav->iCal(Array, -6, NULL), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#5 /usr/share/egroupware/api/src/CalDAV.php(996): calendar_groupdav->get(Array, '13.ics', -6), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#6 /usr/share/egroupware/api/src/WebDAV/Server.php(1278): EGroupware\\Api\\CalDAV->GET(Array), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#7 /usr/share/egroupware/api/src/WebDAV/Server.php(306): HTTP_WebDAV_Server->http_GET(), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#8 /usr/share/egroupware/api/src/CalDAV.php(2058): HTTP_WebDAV_Server->ServeRequest(NULL), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#9 /usr/share/egroupware/groupdav.php(52): EGroupware\\Api\\CalDAV->ServeRequest(), referer: https://.../egroupware/groupdav.php/some-group/calendar/
#10 {main}, referer: https://.../egroupware/groupdav.php/some-group/calendar/
# Instance=default, User=test, Request=GET https://.../groupdav.php/some-group/calendar/13.ics, User-agent=Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0, referer: https://.../egroupware/groupdav.php/some-group/calendar/

Wenn ich die 13.ics im Browser runterlade ist die Datei komplett leer. Some-group hat id 6. Bei dem Termin handelt es sich um einen Geburtstag, also mit Wiederholungstyp jährlich. Wenn ich die Wiederholung ausschalte, klappt alles super und die ics wird gefüllt.

Ab hier unterscheiden sich die wiederkehrenden und nicht wiederkehrenden Events:

github.com

EGroupware/egroupware/blob/a361d05767e725e55d323622144f7d47d9e8e195/calendar/inc/class.calendar_groupdav.inc.php#L684-L691


		if ($this->client_shared_uid_exceptions && $event['recur_type'] && !empty($event['uid']))
		{
			if (is_array($expand))
			{
				if (isset($expand['start'])) $expand['start'] = $this->vCalendar->_parseDateTime($expand['start']);
				if (isset($expand['end'])) $expand['end'] = $this->vCalendar->_parseDateTime($expand['end']);
			}
			$events =& self::get_series($event['uid'], $this->bo, $expand, $user);

Im folgenden code werden die events, welche in variable master enthalten sind, nicht übernommen da isParticipant für eine Gruppe nix zurück liefert (siehe Warning oben). Die Methode gibt dann ein leeres Ergebnis zurück.

github.com

EGroupware/egroupware/blob/a361d05767e725e55d323622144f7d47d9e8e195/calendar/inc/class.calendar_groupdav.inc.php#L800-L804


		if (!$expand && $master && (!$user || self::isParticipant($master, $user)))
		{
			$events = array_merge(array($master), $events);
		}
		return $events;

Weitere werte: user=-6, expand="", master[‘participants’]=Array([9] => ACHAIR) (der User 9 ist Mitglied in Gruppe 6, aber die Gruppe 6 nimmt nicht am Event teil)

Irgendwo auf dem Weg muss dann wohl geprüft werden ob es sich bei user um eine Gruppe handelt.

Grüße
Sascha

RalfBecker 2019-01-21 12:12:19 UTC #4

Hi Sascha,

beides ist jetzt in Git gelöst:

Am einfachsten das ganze File herunter laden und bei dir ersetzen:

https://raw.githubusercontent.com/EGroupware/egroupware/17.1/calendar/inc/class.calendar_groupdav.inc.php

Ralf

derSascha 2019-02-05 08:18:07 UTC #5

Hi Ralf,

ich habe das jetzt noch einmal ausführlich getestet. Bei einem sehr alten (wiederkehrenden) Termin aus 2015 gab es immer noch Probleme (E-Groupware liefert leeren Eintrag zurück womit Davdroid nicht zurecht kommt). Den Termin haben wir jetzt einfach gelöscht. Für die ganzen anderen wiederkehrenden Termine hat dein Bugfix super funktioniert! DAVdroid meldet gerade keine Probleme mehr.

Danke für den schnellen Fix + Grüße
Sascha

RalfBecker 2019-02-05 16:31:37 UTC #6

Ohne die Daten für so einen Termin, kann ich da wenig machen

Das nächste mal die id des Termins ausfindig machen (z.B. aus der URL beim Bearbeiten) und dann mit folgendem Kommando den Termin exportieren:

mysqldump --no-create-info --where="cal_id=<id>" egw_cal egw_cal_dates egw_cal_repeats egw_cal_user

Ralf

Bossi 2019-02-11 21:56:43 UTC #7

Hallo Zusammen,

bin gerade über diesen Threat gestolpert, weil ich exakt das gleiche Problem hatte: Wiederholende Termine werden im Gruppenkalender nicht synchronisiert. Auch bei mir waren die ICS Dateien leer.

Die gepatchte Version habe ich gerade bei mir getestet. Sieht auf den ersten Blick gut aus, die Synchronisierung läuft jetzt sauber durch, und alle Termine werden übertragen.

Danke!

Viele Grüße,

Volker