-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: create samples for the http data proxy flow #8
feat: create samples for the http data proxy flow #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'We are always happy to welcome new contributors ❤️ To make things easier for everyone, please - make sure to follow our contribution guidelines, - check if you have already signed the ECA, and - relate this pull request to an existing issue or discussion.'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor nits, good overall 👍
transfer/transfer-06-http-data-flow/backend-service/build.gradle.kts
Outdated
Show resolved
Hide resolved
...06-http-data-flow/backend-service/src/main/java/org/eclipse/edc/handler/ReceiverHandler.java
Outdated
Show resolved
Hide resolved
...06-http-data-flow/backend-service/src/main/java/org/eclipse/edc/handler/ReceiverHandler.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, a few comments in-line. And could you add the new sample to the CODEOWNERS
file and add @ndr-brt as a responsible there?
Looks good. Please also sign the ECA so then we can merge the PR. |
b9a5136
to
7382321
Compare
chore: squash 4 commits to 1 chore: fix issues related to keystore chore: apply review recommendation chore: apply review recommendation chore: apply review recommendation chore: improve README.md documentation feat: en backend service implementation and add more explaination to the Readme.md feat: en backend service implementation and add more explaination to the Readme.md feat: Add a little http-server module to handle edc.receiver.http.endpoint request WIP: working on documenting the http data flow feat: create and configure a consumer feat: create and configure a provider chore: refactor http-data-flow module name and reorganize the sample module
7382321
to
9a1d825
Compare
Hello @ronjaquensel , the ECA issue had been fixed. |
feat: create samples for the http data proxy flow (eclipse-edc#8) * chore: create base module and add consumer and IT modules * chore: create base module and add consumer and IT modules * chore: squash 12 commits to 1 chore: squash 4 commits to 1 chore: fix issues related to keystore chore: apply review recommendation chore: apply review recommendation chore: apply review recommendation chore: improve README.md documentation feat: en backend service implementation and add more explaination to the Readme.md feat: en backend service implementation and add more explaination to the Readme.md feat: Add a little http-server module to handle edc.receiver.http.endpoint request WIP: working on documenting the http data flow feat: create and configure a consumer feat: create and configure a provider chore: refactor http-data-flow module name and reorganize the sample module feat: done with README.md feat: create http-push-connector chore: update edc version to 1.0.0-milestone-8 and improve documentation chore: fix issues related to keystore chore: apply review recommendation chore: apply review recommendation chore: apply review recommendation chore: improve README.md documentation feat: en backend service implementation and add more explaination to the Readme.md feat: en backend service implementation and add more explaination to the Readme.md
What this PR changes/adds
This PR add a sample to demonstrate a consumer pull data flow between two connectors (on provider and one consumer)
Why it does that
This example shows, step by step, the steps to obtain an asset from the consumer based on a contract with a provider.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #22
Checklist