-
Notifications
You must be signed in to change notification settings - Fork 55
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
docs(edr): Update 07_edrs.md #1646
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.
Just some nits...
Provider Data Plane: | ||
Once the right EDR has been identified using the EDR Management API via the correct asset/agreement/transfer-process that | ||
you want to transfer, we can use the `endpoint` and `authorization` information from the EDRto make the request. The `endpoint` | ||
is used as the Host to which the request is sent and the `authorization` is used as the Token in the authorization header. |
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.
is used as the Host to which the request is sent and the `authorization` is used as the Token in the authorization header. | |
As in the example bellow, the `endpoint` is used as the Host to which the request is sent and the `authorization` replaces TOKEN in the `Authorization` header. |
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.
Fixed with a different proposal.
Co-authored-by: Rafael Magalhães <[email protected]>
Added Rafaels Suggestions
Quality Gate passedIssues Measures |
Even though its not mentioned anywhere in this repo (i thinkg), the name of the pr should adhere to the Eclipse EDC project contributing guidelines: https://github.com/eclipse-edc/docs/blob/main/CONTRIBUTING.md |
WHAT
Fixed the section about provider data plane data transfer in the edr chapter of the management walkthrough and added some more detailed explanation about how the request is build.
WHY
Incorrect example
FURTHER NOTES