csrdelft / csrdelft.nl

De webstek van Civitas Studiosorum Reformatorum, wordt onderhouden door de PubCie.
https://csrdelft.nl
18 stars 11 forks source link

Google contacten fixen #1188

Open NathanHuisman opened 2 months ago

NathanHuisman commented 2 months ago

TODOs:

NathanHuisman commented 2 months ago

https://csrdelft.sentry.io/issues/4985431255/?project=1497101&query=is%3Aunresolved&referrer=issue-stream&stream_index=0

NathanHuisman commented 2 months ago

Probleem is dat het na een poos stopt met werken, de API klaagt dat hij geen OAuth token heeft. WSS dat de refresh token expired is? Als ik de refresh token delete en opnieuw mijn account link doet hij het wel weer

NathanHuisman commented 2 months ago

Ik gok dat hij hier stilletjes geen auth token levert als de refresh token expired is: https://github.com/csrdelft/csrdelft.nl/blob/d1757fd87817685edb6848355fdceb1b3647a2de/lib/service/GoogleContactSync.php#L547

Daardoor klaagt Google later dat ie geen OAuth token heeft.

Moet dit lokaal nog testen, maar daarvoor eerst een client ID fixen