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

refactor: deprecate data-plane-public-api-v2 and proxy feature #4823

Merged

Conversation

ndr-brt
Copy link
Member

@ndr-brt ndr-brt commented Feb 13, 2025

What this PR changes/adds

Deprecates data-plane-public-api-v2 and proxy* attributes in HttpDataAddress

Why it does that

follow decision

Further notes

  • this PR does not adapt the E2E test to use a mocked public API, it will provided in a separate PR
  • found an unused spi module: transfer-data-plane-spi: deleted altogether

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes #4791

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ndr-brt ndr-brt added the refactoring Cleaning up code and dependencies label Feb 13, 2025
@ndr-brt ndr-brt force-pushed the 4791-deprecate-data-plane-public-api branch from b378a4c to 90a87fc Compare February 13, 2025 14:12
Comment on lines +88 to +89
context.getMonitor().warning("The `data-plane-public-api-v2` has been deprecated, please provide an" +
"alternative implementation for Http Proxy if needed");

Check notice

Code scanning / CodeQL

Missing space in string literal Note

This string appears to be missing a space after 'an'.
@ndr-brt ndr-brt force-pushed the 4791-deprecate-data-plane-public-api branch from 90a87fc to f16a821 Compare February 13, 2025 14:43
Copy link
Contributor

@ronjaquensel ronjaquensel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🪓

Copy link
Contributor

@wolf4ood wolf4ood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🪓

@ndr-brt ndr-brt merged commit 1c2dc78 into eclipse-edc:main Feb 13, 2025
21 checks passed
@ndr-brt ndr-brt deleted the 4791-deprecate-data-plane-public-api branch February 13, 2025 15:46
ndr-brt added a commit to Think-iT-Labs/edc-connector that referenced this pull request Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Cleaning up code and dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation of the Http-Proxy data-plane feature
4 participants