eclipse-edc / Connector

EDC core services including data plane and control plane
Apache License 2.0
288 stars 240 forks source link

dataspace protocol: introduce reasons for termination messages #2729

Open juliapampus opened 1 year ago

juliapampus commented 1 year ago

Feature Request

Add a list of termination reasons to TransferTerminationMessage and ContractNegotiationTerminationMessage.

Which Areas Would Be Affected?

spi, protocol services

Why Is the Feature Desired?

According to the spec, both messages support an optional list of termination reasons. Those are of type object. The spec doesn't define the schema.

Solution Proposal

Introduce a class that can be extended if needed (e.g., predefined enums like in StoreFailure):

public class ProtocolServiceFailure extends Failure {

    private final String reason;

    ...
}

Add mapping in dsp transformers.

juliapampus commented 1 year ago

Should be List<Object> reasons for now.

github-actions[bot] commented 1 year ago

This issue is stale because it has been open for 14 days with no activity.

github-actions[bot] commented 1 year ago

This issue was closed because it has been inactive for 7 days since being marked as stale.

github-actions[bot] commented 1 year ago

This issue was closed because it has been inactive for 7 days since being marked as stale.