docs: deprecate DAPS module #4538
Conversation
|
|
||
| In EDC version 0.10.0 we deprecated: | ||
| - module `oauth2-daps` | ||
| - class `Oauth2ServiceImpl` |
There was a problem hiding this comment.
Deprecating DAPS is independent from removing OAuth2 as identity system. If we want to do that, that would require a separate D-R, and we should also remove the controplane-oauth-bom
There was a problem hiding this comment.
I would not remove the oauth2 AFAIK daps module is only an oauth2 extensions with custom decorators. We should probably 🪓 only that
There was a problem hiding this comment.
@paullatzelsperger What are the motivations to keep oauth2 as identity system? I'd go with a single axe blow for both as we want to push toward DCP anyway. I can eventually adapt the D-R accordingly
There was a problem hiding this comment.
its just that both this PR and the decision record are about deprecating DAPS, not Oauth2, the decision was made to axe the earlier, not the latter.
* docs: deprecate DAPS module * pr remark
What this PR changes/adds
Deprecate
oauth2-dapsmodule and releated classes.Why it does that
cleanup
Further notes
Linked Issue(s)
Closes #4534
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.