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.

Sign up for GitHub

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

Jump to bottom

Add oauth2-backend-cats #46

Merged
merged 3 commits into from
Mar 11, 2021
Merged

Add oauth2-backend-cats #46

merged 3 commits into from
Mar 11, 2021

Conversation

bwiercinski
Copy link
Contributor

@bwiercinski bwiercinski commented Mar 9, 2021 •
edited by kubukoz
Loading

Closes #40

  • add Cache abstraction
  • add SttpOauth2ClientCredentialsCatsBackend, which can:
    • fetch token using ClientCredentialsProvider
    • reuse token multiple times using cache
    • fetch new token if the previous expires
    • add Authorization header

@bwiercinski bwiercinski force-pushed the client-credentials-backend branch from dccb2a9 to 05c8e4f Compare March 9, 2021 10:21
@bwiercinski bwiercinski changed the title add oauth2-backend-cats #40 add oauth2-backend-cats Mar 9, 2021
@bwiercinski bwiercinski force-pushed the client-credentials-backend branch from 4844ada to c263a0c Compare March 9, 2021 10:36
@majk-p majk-p requested review from kubukoz, majk-p and tplaskowski March 9, 2021 10:39
@kubukoz kubukoz changed the title #40 add oauth2-backend-cats Add oauth2-backend-cats Mar 9, 2021
kubukoz
kubukoz requested changes Mar 9, 2021
View reviewed changes
build.sbt Outdated Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Outdated Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Outdated Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Outdated Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Outdated Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Show resolved Hide resolved
...n/scala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackend.scala Show resolved Hide resolved
...ala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackendSpec.scala Show resolved Hide resolved
...ala/com/ocadotechnology/sttp/oauth2/backend/SttpOauth2ClientCredentialsCatsBackendSpec.scala Show resolved Hide resolved
@bwiercinski bwiercinski mentioned this pull request Mar 10, 2021
Open
@bwiercinski bwiercinski mentioned this pull request Mar 10, 2021
Closed
@kubukoz kubukoz merged commit a404b5c into polyvariant:main Mar 11, 2021
@bwiercinski bwiercinski deleted the client-credentials-backend branch March 16, 2021 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@majk-p majk-p majk-p approved these changes

@kubukoz kubukoz kubukoz approved these changes

@tplaskowski tplaskowski Awaiting requested review from tplaskowski

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Client credentials backend (interceptor) module
3 participants
@bwiercinski @kubukoz @majk-p