Skip to content

Conversation

Dindexx
Copy link
Contributor

@Dindexx Dindexx commented Apr 24, 2025

Short description of what this resolves:

Implements Relying-Party(RP) authentication based on ARF:

https://bmi.usercontent.opencode.de/eudi-wallet/eidas-2.0-architekturkonzept/flows/Wallet-Relying-Party-Authentication/

Changes proposed in this pull request:

  • Adds bunch of business logic for validating RP trust level
  • Adds IRpAuthService with one signature
  • Adds RP validation for RequestObject in AuthRequestService

kenkosmowski and others added 9 commits April 11, 2025 10:08
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
…tificate

# Conflicts:
#	src/WalletFramework.Oid4Vc/Oid4Vp/Dcql/Models/CredentialQuery.cs
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: kenkosmowski <ken.kosmowski@gmx.de>
Signed-off-by: Kevin <kevin.dinh@lissi.id>
@Dindexx Dindexx self-assigned this Apr 24, 2025
@Dindexx Dindexx requested a review from JoTiTu April 24, 2025 15:50
Copy link
Contributor

@JoTiTu JoTiTu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

[Fact]
public void Untrusted_Access_Certificate_results_in_Trust_Level_Abort()
{
// TODO: Implement this test
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May add these tests if feasible

@Dindexx Dindexx merged commit cca9fda into main Apr 28, 2025
3 checks passed
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.

3 participants