search

math-GH / be_piwikcharts

Extension for Open Source Content Management System "Contao": Displays an overview about figures of Piwik-Installation
https://contao.org/de/extension-list/fileview/be_piwikcharts.html
2 stars 2 forks source link

Update bepiwikcharts.php #10

Closed kroerig closed 3 months ago

kroerig commented 5 months ago

Übergabe des API Tokens per GET soweit möglich entfernt. Bei den Graphen ist noch Handlungsbedarf. // QnD: Der BuildURL Aufruf bekommt den Token mit übergeben.

math-GH commented 4 months ago

Hi, sorry für die späte Rückmeldung. Danke für den PR. Ich schaue ihn mir an.

Was bedeutet QnD?

Was meinst du mit "Bei den Graphen ist noch Handlungsbedarf."? Willst du da noch etwas hinzufügen oder ist der PR erstmal so ready für dich?

kroerig commented 4 months ago

QnD: Quick and Dirty.

Ich habe spontan nicht herausgefunden, ob und wie sich die Graphen per POST-Request abrufen lassen. Deswegen haben ich hier den GET-Aufruf belassen und nur um den Tokenparameter ergänzt. Alle anderen Daten kommen jetzt über POST-Requests.

Das bedeutet aber, das in aktuellen Matomo-Installationen das Token für "unsichere" (GET-) Requests zugelassen werden muss.