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

Remove SKI/AKI usage #805

Open
5 tasks
SebastianOpriel opened this issue Feb 15, 2024 · 1 comment
Open
5 tasks

Remove SKI/AKI usage #805

SebastianOpriel opened this issue Feb 15, 2024 · 1 comment
Assignees
Labels
kind/enhancement New feature or request scope/ce sovity's Open Source Community Edition

Comments

@SebastianOpriel
Copy link
Member

SebastianOpriel commented Feb 15, 2024

Feature Request

Description

SKI/AKI is a relict of from the first days of IDS. We want to discuss, if the SKI/AKI can be removed and replaced with a custom ID coming from a data space authority in control of DAPS. For those tagged, please (if you can) edit this issue and add information on where and how SKI/AKI is used.

Which Areas Would Be Affected?

KC DAPS

Management CLI uses SKI/AKI by default to generate client ID, but can be overridden.

EDC

  • SKI/AKI is used via EDC_OAUTH_CLIENT_ID field
  • EDC_OAUTH_CLIENT_ID is used to receive a DAT from DAPS, who compares this ID with Subject field.
  • something about which field is used for policies? @efiege
    • The Referring-Connector-Policy will check the referringConnector-Claim of the DAT. However this does not affect the connector implementation, but the configuration of the DAPS. DAPS implementations which are not capable of setting the referringConnector-Claim may be incompatible afterwards. @SebastianOpriel

Broker

At its base, the Broker is an EDC connector which means that same rules apply. A client ID is needed to register the Broker in DAPS, but it's not generated in the Broker itself but rather provided by the host via environment variables.
Since the clientID is just an identifier for DAPS, dropping the AKI/SKI generation method should not affect the Broker in any way

Clearing House

We are currently exploring replacing the SKI/AKI as the client_id in the clearing-house-server with the participantId. The clearing-house-server, if possible, should no longer depend on the SKI/AKI.

Authority Portal

In the AP, we use AKI and SKI to generate the client ID (for registering the connector at the DAPS) of self-hosted connectors. Since we already use a different method (deriving it from the connector ID/participant ID) to generate client IDs for CaaS connectors, which is also applicable to self-hosted connectors, this would be a very quick adjustment.

Addition (April 9th): We use the client ID (AKI/SKI), which is stored in our database, to check if a connector with the given certificate is already registered. So in some form, this information would still be needed to be saved, though not used as the client ID.

Why Is the Feature Desired?

SKI/AKI is always an issue. The overall trust chain might simply rely on another ID, as SKI/AKI coming from the X509 certificate is not used for checks.

Solution Proposal and Work Breakdown

EDC

  • Remove mandatory field EDC_OAUTH_CLIENT_ID and by default preset it with MY_EDC_PARTICIPANT_ID
  •  Remove documentation about SKI/AKI

Authority Portal

@SebastianOpriel SebastianOpriel added kind/enhancement New feature or request scope/ce sovity's Open Source Community Edition task/analyze Need for investigation status/blocked/needs-product requires input from product owner labels Feb 15, 2024
@SebastianOpriel SebastianOpriel added status/blocked/needs-info requires more information in order to be completed and removed status/blocked/needs-product requires input from product owner labels Feb 15, 2024
@SebastianOpriel SebastianOpriel changed the title Remove SKI/AKI from component architecture Discuss removal of SKI/AKI from component architecture Feb 15, 2024
@dhommen
Copy link
Contributor

dhommen commented Feb 16, 2024

We are currently exploring replacing the SKI/AKI as the client_id in the clearing-house-server with the participantId. The clearing-house-server, if possible, should no longer depend on the SKI/AKI.

@AbdullahMuk AbdullahMuk added the clean-backlog requires backlog cleaning label May 2, 2024
@SebastianOpriel SebastianOpriel changed the title Discuss removal of SKI/AKI from component architecture Remove SKI/AKI usage Sep 26, 2024
@SebastianOpriel SebastianOpriel removed task/analyze Need for investigation status/blocked/needs-info requires more information in order to be completed clean-backlog requires backlog cleaning labels Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request scope/ce sovity's Open Source Community Edition
Projects
None yet
Development

No branches or pull requests

8 participants