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

Add oauth2 Client Credentials Flow #457

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

knoppiks
Copy link
Collaborator

@knoppiks knoppiks commented Dec 9, 2024

No description provided.

@knoppiks knoppiks self-assigned this Dec 9, 2024
@knoppiks knoppiks linked an issue Dec 9, 2024 that may be closed by this pull request
@knoppiks knoppiks force-pushed the 224-add-openid-connect-client-credentials-flow branch from d098051 to d0ed984 Compare December 9, 2024 13:21
Copy link

codecov bot commented Dec 9, 2024

Codecov Report

Attention: Patch coverage is 97.22222% with 2 lines in your changes missing coverage. Please review.

Project coverage is 96.15%. Comparing base (77b8166) to head (35d3194).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...care/smith/fts/util/auth/HttpServerOAuth2Auth.java 91.30% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #457      +/-   ##
============================================
- Coverage     96.18%   96.15%   -0.03%     
  Complexity      415      415              
============================================
  Files           108      111       +3     
  Lines          1545     1611      +66     
  Branches         42       43       +1     
============================================
+ Hits           1486     1549      +63     
- Misses           42       44       +2     
- Partials         17       18       +1     
Files with missing lines Coverage Δ
...fts/tca/consent/FhirConsentedPatientsProvider.java 100.00% <100.00%> (ø)
...n/java/care/smith/fts/util/AgentConfiguration.java 100.00% <100.00%> (ø)
...ain/java/care/smith/fts/util/HttpClientConfig.java 100.00% <100.00%> (ø)
...ain/java/care/smith/fts/util/WebClientFactory.java 97.05% <100.00%> (-2.95%) ⬇️
.../java/care/smith/fts/util/auth/HttpClientAuth.java 100.00% <100.00%> (ø)
...care/smith/fts/util/auth/HttpClientOAuth2Auth.java 100.00% <100.00%> (ø)
...care/smith/fts/util/auth/HttpServerAuthConfig.java 100.00% <100.00%> (ø)
...care/smith/fts/util/auth/HttpServerAuthMethod.java 100.00% <100.00%> (ø)
.../util/auth/OAuth2ConfigurationExistsCondition.java 100.00% <100.00%> (ø)
...care/smith/fts/util/auth/HttpServerOAuth2Auth.java 91.30% <91.30%> (ø)

@knoppiks knoppiks force-pushed the 224-add-openid-connect-client-credentials-flow branch from d0ed984 to 17e4c9c Compare December 11, 2024 16:36
@knoppiks knoppiks force-pushed the 224-add-openid-connect-client-credentials-flow branch from 17e4c9c to c67aef0 Compare January 8, 2025 07:39
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 12 times, most recently from c82a3a1 to 0ba3bc9 Compare January 10, 2025 13:59
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 10 times, most recently from c57576c to b78b578 Compare January 23, 2025 11:00
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 7 times, most recently from 8271935 to 15ca3b9 Compare January 23, 2025 12:56
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch from 15ca3b9 to 62194ab Compare January 24, 2025 12:16
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch from 62194ab to 35d3194 Compare January 24, 2025 13:08
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.

Add OpenID Connect Client Credentials Flow
2 participants