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

feat: move STS modules into IdentityHub #4839

Conversation

paullatzelsperger
Copy link
Member

@paullatzelsperger paullatzelsperger commented Feb 25, 2025

What this PR changes/adds

moves all STS modules (Except the remote client) into IdentityHub:

  • remove STS API
  • remove STS Accounts API
  • Remove STS Server launcher
  • remove STS client configuration module
  • remove sts core module
  • move SQL sts client store

Why it does that

STS is always embedded into IdentityHub

Further notes

Who will sponsor this feature?

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

Linked Issue(s)

Closes # <-- insert Issue number if one exists

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

@paullatzelsperger paullatzelsperger added the refactoring Cleaning up code and dependencies label Feb 25, 2025
@paullatzelsperger paullatzelsperger force-pushed the feat/move_sts_modules_into_ih branch from 721334c to 20cb33e Compare February 25, 2025 09:04
@paullatzelsperger paullatzelsperger force-pushed the feat/move_sts_modules_into_ih branch from 20cb33e to 0d13b55 Compare February 25, 2025 09:05
@paullatzelsperger paullatzelsperger added the breaking-change Will require manual intervention for version update label Feb 25, 2025
@paullatzelsperger paullatzelsperger marked this pull request as ready for review February 25, 2025 09:45
@paullatzelsperger paullatzelsperger merged commit a69e955 into eclipse-edc:main Feb 25, 2025
1 check passed
@paullatzelsperger paullatzelsperger deleted the feat/move_sts_modules_into_ih branch February 25, 2025 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Will require manual intervention for version update refactoring Cleaning up code and dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants