Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.13.32 added optional parameters req, res for getContacts, createCon… #74

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

siavashcsr
Copy link
Collaborator

…tct, updateContact and deleteContact

Wir haben folgenden Fall: Das CRM "Exact Online" verwendet OAuth und hat RefreshToken und AccessToken. Wir müssen vor dem Aufrufen deren API immer einen neuen AccessToken holen. Wir kiregen als Antwort aber nicht nur einen Access Token, sondern auch einen neuen RefreshToken zurück. Es verfällt dann leider aber auch das alte RefreshToken. Deswegen schlägt dann auch der anschließende Aufruf von getToken fehl, der im integration-bridge-framework gemacht wird. D.h. wir können den Refresh Token nicht erneuern. Daher haben wir mal das Interface von getContacts, updateContact createContact und deleteContact erweitert um request und response Objekt. Auf diese Art können wir den refreshToken direkt selber mittels "res.header("X-Provider-Key", apiKey);" setzen. Denn: wir kriegen ihn ja eh, wenn wir uns einen neuen Acces Token holen.

War jetzt bestimmt schwer verständlich alles. Können wir Morgen im Integrations Meeting zusammen durchgehen.
Danke und Grüße
Sia

@siavashcsr siavashcsr self-assigned this Mar 20, 2023
@siavashcsr
Copy link
Collaborator Author

Doch nicht notwendig, haben ein Lösung in der Integration gefunden, die keine Modifikation braucht

@siavashcsr siavashcsr closed this Mar 21, 2023
@siavashcsr siavashcsr reopened this Mar 27, 2023
@siavashcsr
Copy link
Collaborator Author

siavashcsr commented Mar 27, 2023

Wir hatten eine andere Lösung in Betracht gezogen. Hat nicht hingehauen. Daher hab ich den Pull Request wieder aufgemacht.

@siavashcsr siavashcsr force-pushed the interface_extension branch from 4782cef to 67224f4 Compare March 27, 2023 18:56
@siavashcsr siavashcsr changed the title 0.13.27 added optional parameters req, res for getContacts, createCon… 0.13.32 added optional parameters req, res for getContacts, createCon… Mar 27, 2023
@siavashcsr siavashcsr force-pushed the interface_extension branch from 67224f4 to 7e85914 Compare March 27, 2023 19:17
@siavashcsr siavashcsr removed the request for review from fgladisch March 27, 2023 19:19
@LemonaOna
Copy link
Contributor

Grundsätzlich machbar, das so zu erweitern, ich frage mich aber, ob das das Problem löst, dass sich der Key der Integration dann ja ständig ändert und der Cache-key sich somit immer ändert und die Desktop Clients immer neu rendern. Aber vllt durchblick ich die Änderungen auch nicht ganz. Lass uns morgen mal drüber reden. Ich stell einen Termin ein.

@siavashcsr
Copy link
Collaborator Author

Ja du hast recht, das wird das Problem nicht lösen. Die Intention hier, ist ein anderes Problem: im CRM Exact ist ein refresh Token nur einmal nutzbar und dummerweise bekommt man in demselben API Call ein neues refresh token als auch ein neues acces token. D.h. es wird getContacts aufgerufen, dort machen wir einen API Call um ein neues acces token zu bekommen und bekommen im selben Aufruf auch ein neues refresh token mit gegeben. Wenn dann im nächsten Schritt getToken aufgerufen wird, können wir kein neues refresh token mehr von der API holen, da das alte refresh token invalidiert wurde. D.h. die Änderungen zielen darauf ab, dass wir bereits beim Aufruf von getContacts das neue refresh token zurück geben können. Aber wir können das noch im Meeting diskutieren.

@LemonaOna LemonaOna requested a review from a team May 31, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants