sovity / edc-ui

Connector UI for the sovity EDC Community Edition.
https://sovity.de/en/connect-to-data-space-en/
Apache License 2.0
33 stars 11 forks source link

HTTP Data Transfer: Remove EDC prefix for headers in data address property #706

Closed bearn01d closed 3 months ago

bearn01d commented 3 months ago

This resolves #702 by not prefixing headers for the HTTP data transfer with the EDC namespace as the EDC expects them without, see https://github.com/eclipse-edc/Connector/blob/5803513f0c4cc795c0d1d069f7039c8ca1bd8f7e/spi/common/core-spi/src/main/java/org/eclipse/edc/spi/types/domain/HttpDataAddress.java#L131.

### Checklist
- [x] The PR title is short and expressive.
- [x] I have updated the CHANGELOG.md. See [changelog_update.md](https://github.com/sovity/authority-portal/tree/main/docs/dev/changelog_updates.md) for more information.
- [x] I have performed a **self-review**
tmberthold commented 3 months ago

@richardtreier just as note: there are some workflows in the edc-ui repo, which trigger only for pushes to branches within this repo. This PR is coming from a fork and thus not a branch within the repo, so some actions are not triggered, e.g: https://github.com/sovity/edc-ui/blob/89b35e86c28b422b3b09399245b531f6fac1b870/.github/workflows/test_angular_project.yaml#L4