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

Transfer fails if method-parametrization is enabled #703

Closed
tmberthold opened this issue Mar 18, 2024 · 5 comments
Closed

Transfer fails if method-parametrization is enabled #703

tmberthold opened this issue Mar 18, 2024 · 5 comments
Assignees
Labels
component/edc-ui affected software component kind/bug Something isn't working. The software does not behave as expected or specified. scope/mds related to MDS

Comments

@tmberthold
Copy link
Member

tmberthold commented Mar 18, 2024

Description - What happened? *

As soon as the method-parameterization is enabled for an asset and the asset is created, every later on transfer fails with an log-error-message at the provider when the asset is to be transferred.

Expected Behavior *

No error message.

Observed Behavior *

Error message in the logs, see GIF how to reproduce.

Steps to Reproduce

I'd recommend looking at the GIF below, this is clearer than what we got:

  1. Create an asset and enable parametrization for method, path, query param, and body.
  2. Create contract definition, negotiate a contract on another connector, and go to transfer.
  3. Fill in all the details for the data sink and also the method, path, parameters, and body for parametrization.
  4. Initiate transfer.

Actual result:
Transfer status is "Requested" in EDC UI.
Transfer fails with error in logs:
2024-03-17 23:52:17 TransferProcess: ID d4d60f54-bc4f-418b-91d3-f9e20d7c5c02. Fatal error while Initiate data flow. Error details: Failed to delegate data transfer to Data Plane: Failed to build HttpDataSource: DataFlowRequest 2e417d75-e77e-472a-9c1e-d505e82425fa: 'method' property is missing

Context Information

We think it could be caused by the way the UI saves the Assets when method-parameterization is enabled.

Stakeholder

@ip312 @jkbquabeck

Relevant log output

Fatal error while Initiate data flow. Error details: Failed to delegate data transfer to Data Plane: Failed to build HttpDataSource: DataFlowRequest (...): 'method' property is missing

Screenshots

reproduce (1)

@tmberthold tmberthold added the kind/bug Something isn't working. The software does not behave as expected or specified. label Mar 18, 2024
@AbdullahMuk AbdullahMuk added this to the Release March 22 milestone Mar 18, 2024
@AbdullahMuk AbdullahMuk added component/edc-ui affected software component scope/mds related to MDS labels Mar 18, 2024
@AbdullahMuk AbdullahMuk assigned ununhexium and unassigned illfixit Mar 20, 2024
@AbdullahMuk
Copy link
Collaborator

@tmberthold did MDS provide the steps to reproduce?

/cc @ununhexium @illfixit

@tmberthold
Copy link
Member Author

tmberthold commented Mar 21, 2024

@tmberthold did MDS provide the steps to reproduce?

I'd recommend looking at the GIF, this is clearer than what we got:

Steps to reproduce:

  1. Create an asset and enable parametrization for method, path, query param, and body.
  2. Create contract definition, negotiate a contract on another connector, and go to transfer.
  3. Fill in all the details for the data sink and also the method, path, parameters, and body for parametrization.
  4. Initiate transfer.

Actual result:
Transfer status is "Requested" in EDC UI.
Transfer fails with error in logs:
2024-03-17 23:52:17 TransferProcess: ID d4d60f54-bc4f-418b-91d3-f9e20d7c5c02. Fatal error while Initiate data flow. Error details: Failed to delegate data transfer to Data Plane: Failed to build HttpDataSource: DataFlowRequest 2e417d75-e77e-472a-9c1e-d505e82425fa: 'method' property is missing

@ununhexium
Copy link
Collaborator

ununhexium commented Mar 25, 2024

We need a custom version of the Eclipse EDC to backport a patch from V0.5.1 to V0.2.1, which we will do ASAP.

The bug should be fixed by using this patch.

This is the same kind of change as PMO 694.

@AbdullahMuk AbdullahMuk changed the title Transfer fails if method-parameterization is enabled Transfer fails if method-parametrization is enabled Mar 28, 2024
@SebastianOpriel
Copy link
Member

For general error handling, in case a provider has an internal error that prevents him to send a proper response, see sovity/edc-ce#688

@ununhexium
Copy link
Collaborator

Fixed in EDC CE 7.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/edc-ui affected software component kind/bug Something isn't working. The software does not behave as expected or specified. scope/mds related to MDS
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

5 participants