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

Fix for issue #59 #60

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix for issue #59 #60

wants to merge 3 commits into from

Conversation

g0uZ
Copy link

@g0uZ g0uZ commented Dec 27, 2022

Adapt code to recent API modification done by Mercedes.

Anti csrf token as stated here : https://developer.mercedes-benz.com/content-page/oauth-documentation#_from_the_developers_perspective

state=<insert_client_state_here>: An opaque value used by the client to maintain state between the request and callback. The authorization server includes this value when redirecting the user-agent back to the client. The parameter SHOULD be used for preventing cross-site request forgery.
@maltenorskov
Copy link

I modified the files manually, and I can confirm that it works :)

@RAiN3RWAhN5iNN
Copy link

I also can confirm it works :-)

@jamrogers
Copy link

I modified the files manually, I have no idea what I'm doing but I got it working

@fr3nch13
Copy link

Can we please push this pull request?
When trying to setup the plugin, I get an invalid_scope error because it requires openid.

@fr3nch13
Copy link

Update. I applied this patch, and I'm still getting the invalid_scope error.

@flopro1
Copy link

flopro1 commented Apr 15, 2023

@jamrogers @maltenorskov @RAiN3RWAhN5iNN How can I manually update the files to test it?

@Jeepersfinest
Copy link

Jeepersfinest commented Apr 26, 2023

I also can confirm it works :-)

Update: Unfortunately only for 2 hours. Token refresh will not works, Error 400

Any further ideas?

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.

7 participants