simplesamlphp / simplesamlphp-module-oidc

A SimpleSAMLphp module for OIDC OP support.
Other
46 stars 23 forks source link

Add Request Object support #245

Closed cicnavi closed 2 months ago

cicnavi commented 2 months ago

As per: https://openid.net/specs/openid-connect-core-1_0.html#RequestObject

This is optional to implement in Core specification, but is required to support automatic client registration from Federation spec.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 55.59701% with 119 lines in your changes missing coverage. Please review.

Project coverage is 52.84%. Comparing base (483d954) to head (90183b7).

Files with missing lines Patch % Lines
src/Services/Container.php 0.00% 24 Missing :warning:
src/Factories/RequestRulesManagerFactory.php 0.00% 19 Missing :warning:
src/Forms/ClientForm.php 33.33% 18 Missing :warning:
src/Server/Grants/AuthCodeGrant.php 0.00% 10 Missing :warning:
src/Services/DatabaseMigration.php 0.00% 9 Missing :warning:
src/Server/AuthorizationServer.php 0.00% 6 Missing :warning:
src/Server/RequestRules/Rules/MaxAgeRule.php 0.00% 6 Missing :warning:
src/Server/RequestRules/Rules/PromptRule.php 0.00% 6 Missing :warning:
src/Controller/Federation/Test.php 0.00% 5 Missing :warning:
src/Server/Grants/OAuth2ImplicitGrant.php 0.00% 5 Missing :warning:
... and 6 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## wip-version-6 #245 +/- ## =================================================== + Coverage 51.93% 52.84% +0.91% - Complexity 1134 1155 +21 =================================================== Files 121 122 +1 Lines 4242 4320 +78 =================================================== + Hits 2203 2283 +80 + Misses 2039 2037 -2 ```

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