-
Notifications
You must be signed in to change notification settings - Fork 23
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
Enhance Subscription Provider Endpoint to Include EDC Configuration Details #841
Comments
relates to eclipse-tractusx/sig-release#731 |
@Phil91 can you please check once how the iatp and issuer id is getting created as part of GET /api/administration/companydata/decentralidentity/urls? ideally we use the same for this endpoint. |
Phil91
added a commit
that referenced
this issue
Jul 24, 2024
8 tasks
Phil91
added a commit
that referenced
this issue
Jul 25, 2024
Phil91
added a commit
that referenced
this issue
Jul 25, 2024
Phil91
added a commit
that referenced
this issue
Jul 25, 2024
#867) Refs: #841 Co-authored-by: Norbert Truchsess <[email protected]> Reviewed-by: Norbert Truchsess <[email protected]>
@sachinargade123 @ciprianherciu it's ready for testing |
Phil91
added a commit
that referenced
this issue
Jul 26, 2024
#867) Refs: #841 Co-authored-by: Norbert Truchsess <[email protected]> Reviewed-by: Norbert Truchsess <[email protected]>
ntruchsess
added a commit
that referenced
this issue
Jul 29, 2024
#867) Refs: #841 Co-authored-by: Norbert Truchsess <[email protected]> Reviewed-by: Norbert Truchsess <[email protected]>
ntruchsess
added a commit
that referenced
this issue
Jul 29, 2024
* fix(apps): remove leading '/' from the activate subscription endpoint (#861) Refs: #856 * bug(offersubscription): suppress activation of external serviceaccounts in keycloak on subscription-activation (#864) Refs: #856 * fix(dim): fix callback logic for dim requests (#863) Refs: #862 chore(dependencies): bump efcore to 8.0.7 (#857) * increase efcore version to latest 8.0.7 to resolve security-issue in transitive dependency in System.Text.Json * fix(offersubscription): skip optional autosetupprovidercallback if it is not configured (#865) * add clientPrefix to workers appsettings (#870) * fix: enhance subscription/provider endpoint with external Service data (#867) Refs: #841 * fix(sdDoc): set process step to skipped if ClearinghouseConnectDisabled is true (#874) Refs: #792 * build(deps): bump MimeKit from 4.3.0 to 4.7.1 (#833) eclipse-tractusx/portal#369 * build(deps): bump MimeKit from 4.3.0 to 4.7.1 in /tests/endtoend Bumps MimeKit from 4.3.0 to 4.7.1. --- updated-dependencies: - dependency-name: MimeKit dependency-type: direct:production ... * chore: upgrade mimekit in sendmail project * chore: update dependencies file --------- * upgrade implicit dependencies on System.Text.Json (#875) * Serilog.Settings.Configuration to 8.0.2 * remove redundant dependencies on Microsoft.Extensions.Hosting * bump framework version to 2.4.2 after merge of main --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Norbert Truchsess <[email protected]> Co-authored-by: Phil Schneider <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Evelyn Gurschler <[email protected]> Reviewed-By: Evelyn Gurschler <[email protected]> Reviewed-By: Norbert Truchsess <[email protected]> Reviewed-By: Phil Schneider <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description:
Our current system, which integrates Self-Sovereign Identity (SSI) and Enterprise Data Centers (EDCs), lacks a crucial feature that allows app/service providers who manage an EDC on behalf of a customer, to verify that the EDC is accurately configured to the customer's wallet rather than the service/app provider's wallet. To address this, we need to enhance the endpoint
GET /api/apps/{appId}/subscription/{subscriptionId}/provider
to return additional configuration details.Acceptance Criteria:
externalService
.externalService
section should contain the following fields with their respective data types:trusted_issuer
(string)participant_id
(string)iatp_id
(string)did_resolver
(string)decentralIdentityManagementAuthUrl
(string)decentralIdentityManagementServiceUrl
(string)*
"trusted_issuer"
iatp address of the issuer (e.g. did:web:dim-static-prod.dis-cloud-prod.cfapps.eu10-004.hana.ondemand.com:dim-hosted:2f45795c-d6cc-4038-96c9-63cedc0cd266:holder-iatp)*
"participant_id"
Subscription customer BPNL*
"iatp_id"
iatp address of the holder (e.g. did:web:dim-static-prod.dis-cloud-prod.cfapps.eu10-004.hana.ondemand.com:dim-hosted:2f45795c-d6cc-4038-96c9-63cedc0cd266:holder-iatp)*
"did_resolver"
URL of the didResolver available inside the env. variables (e.g. https://bpn-did-resolution-service.int.demo.catena-x.net/api/directory)*
"DecentralIdentityManagementAuthURL"
is configured inside the environment variables and displays the DIM e.g.https://dis-integration-service-prod.eu10.dim.cloud.sap/api/v2.0.0/iatp/catena-x-portal*
"DecentralIdentityManagementServiceURL"
should be fetched from the portal backend which displays the customer wallet instance url - e.g. https://bpnl0000000001j2-j2.authentication.eu10.hana.ondemand.com/externalService
section with an explanation of each field.Tasks:
externalService
section.Proposed new response body details:
Test Case 1: Retrieve Subscription Provider with EDC Configuration Details
Objective: Ensure the endpoint returns the enhanced response including the
externalService
section.Steps:
/api/apps/{appId}/subscription/{subscriptionId}/provider
.externalService
section.externalService
fields contain expected data types and values.Expected Result: The response includes the
externalService
section with correct details fetched from the subscription customer's configuration.Test Case 2: Validate Field Data Types in
externalService
Objective: Confirm that all fields in the
externalService
section have the correct data types.Steps:
/api/apps/{appId}/subscription/{subscriptionId}/provider
.externalService
section is of type string.Expected Result: All fields in the
externalService
response are strings.Test Case 3: Backward Compatibility
Objective: Ensure that the enhancement does not break existing functionality.
Steps:
/api/apps/{appId}/subscription/{subscriptionId}/provider
using an API version that predates the enhancement.externalService
section.Expected Result: The response is as per the original schema and functionality remains unaffected for the old API version.
Test Case 4: Error Handling for Non-Existent Subscription
Objective: Verify that the endpoint handles requests for non-existent subscriptions correctly.
Steps:
/api/apps/{appId}/subscription/{nonExistentSubscriptionId}/provider
.Expected Result: The endpoint returns a 404 Not Found for non-existent subscriptions.
Test Case 5: Security and Authorization Check
Objective: Ensure that only authorized users can access the EDC configuration details.
Steps:
/api/apps/{appId}/subscription/{subscriptionId}/provider
without proper authorization.Expected Result: Unauthorized users cannot retrieve EDC configuration details from the endpoint.
Test Case 6: Documentation and Schema Validation
Objective: Confirm that the endpoint documentation has been updated and the schema is valid.
Steps:
externalService
section and its fields.Expected Result: The documentation accurately reflects the endpoint's updated schema and functionality.
Test Case 7: Field Presence Validation
Objective: Check that all new fields in the
externalService
section are always present in the response.Steps:
/api/apps/{appId}/subscription/{subscriptionId}/provider
.externalService
section are present, regardless of whether they are populated or null.Expected Result: The response always contains the
externalService
section with all expected fields.The text was updated successfully, but these errors were encountered: