jenkinsci / oic-auth-plugin

A Jenkins plugin which lets you login to Jenkins using your own, self-hosted or public openid connect server.
https://plugins.jenkins.io/oic-auth
MIT License
74 stars 94 forks source link

Replace EOL Google Oauth library, disable nonce check during token refresh #419

Closed jtnord closed 1 month ago

jtnord commented 1 month ago

This is #409 with 2 additional commits.

Testing done

Submitter checklist

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 69.30693% with 93 lines in your changes missing coverage. Please review.

Project coverage is 72.20%. Comparing base (b48a299) to head (fb4ce0c). Report is 10 commits behind head on master.

Files with missing lines Patch % Lines
...va/org/jenkinsci/plugins/oic/OicSecurityRealm.java 75.48% 26 Missing and 12 partials :warning:
...i/plugins/oic/OicServerWellKnownConfiguration.java 53.48% 15 Missing and 5 partials :warning:
...kinsci/plugins/oic/AnythingGoesTokenValidator.java 66.66% 7 Missing :warning:
...jenkinsci/plugins/oic/CustomOidcConfiguration.java 65.00% 4 Missing and 3 partials :warning:
...nsci/plugins/oic/OicServerManualConfiguration.java 77.41% 5 Missing and 2 partials :warning:
...insci/plugins/oic/ProxyAwareResourceRetriever.java 70.58% 3 Missing and 2 partials :warning:
...nsci/plugins/oic/ssl/AnythingGoesTrustManager.java 37.50% 4 Missing and 1 partial :warning:
...n/java/org/jenkinsci/plugins/oic/ssl/TLSUtils.java 50.00% 2 Missing and 1 partial :warning:
...nsci/plugins/oic/ssl/IgnoringHostNameVerifier.java 50.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #419 +/- ## ============================================ - Coverage 74.16% 72.20% -1.96% + Complexity 261 199 -62 ============================================ Files 15 16 +1 Lines 1022 878 -144 Branches 147 119 -28 ============================================ - Hits 758 634 -124 + Misses 191 182 -9 + Partials 73 62 -11 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.