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

Merged
merged 1 commit into from
Jan 29, 2025

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 98.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 96.24%. Comparing base (7fd18b1) to head (87073ec).
Report is 2 commits behind head on main.

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

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #457      +/-   ##
============================================
+ Coverage     96.18%   96.24%   +0.06%     
  Complexity      415      415              
============================================
  Files           108      111       +3     
  Lines          1545     1599      +54     
  Branches         42       43       +1     
============================================
+ Hits           1486     1539      +53     
- Misses           42       43       +1     
  Partials         17       17              
Files with missing lines Coverage Δ
...fts/tca/consent/FhirConsentedPatientsProvider.java 100.00% <100.00%> (ø)
...a/consent/configuration/GicsFhirConfiguration.java 100.00% <ø> (ø)
...uration/GpasFhirDeIdentificationConfiguration.java 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 100.00% <100.00%> (ø)
.../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%> (ø)
... and 2 more

@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 8 times, most recently from 7d1037a to 2864fc0 Compare January 27, 2025 13:14
.github/workflows/build.yml Outdated Show resolved Hide resolved
clinical-domain-agent/application.yaml Outdated Show resolved Hide resolved
clinical-domain-agent/pom.xml Outdated Show resolved Hide resolved
clinical-domain-agent/pom.xml Outdated Show resolved Hide resolved
docs/configuration/security/oauth2.md Outdated Show resolved Hide resolved
docs/types/HttpClientConfig.md Outdated Show resolved Hide resolved
@knoppiks knoppiks removed their assignment Jan 27, 2025
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 5 times, most recently from ac9b8db to 2815739 Compare January 28, 2025 13:03
renovate.json Outdated Show resolved Hide resolved
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 5 times, most recently from 2224b1d to 704a3f3 Compare January 29, 2025 10:28
@knoppiks knoppiks marked this pull request as ready for review January 29, 2025 10:34
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch 3 times, most recently from 2ce4fac to d6c341e Compare January 29, 2025 11:51
@trobanga trobanga force-pushed the 224-add-openid-connect-client-credentials-flow branch from d6c341e to 87073ec Compare January 29, 2025 12:11
@knoppiks knoppiks merged commit 659e812 into main Jan 29, 2025
32 of 33 checks passed
@knoppiks knoppiks deleted the 224-add-openid-connect-client-credentials-flow branch January 29, 2025 12:36
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