-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
[16.0] [IMP] webservice: add support for oauth2 #35
Conversation
Hi @etobella, |
618b2c3
to
6061dea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need tests
changing to draft: this implementation requests a new token for each api call, and some OAuth2 provider have an invoicing per token -> not acceptable. |
4c6b99d
to
990c972
Compare
372e70b
to
dc970fd
Compare
ae82a36
to
fc2accf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overall
responses.add(responses.GET, f"{self.url}endpoint", body="OK") | ||
|
||
with mock_cursor(self.env.cr): | ||
result = self.webservice.call("get", url=f"{self.url}endpoint") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to test that commit has been called? Or that rollback is handled in case of errors?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e7bf9a1
to
ee889c4
Compare
ee889c4
to
1185d0f
Compare
b20c9e0
to
4b5a907
Compare
Allow using oauth2 with Backend Application Flow / Client Credentials Grant.
4b5a907
to
cfad24a
Compare
/ocabot merge minor |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 0524d1b. Thanks a lot for contributing to OCA. ❤️ |
No description provided.