DSP catalog request should not return 401 when trusted-issuer check fails #5487
Replies: 2 comments 2 replies
-
|
Thanks for your contribution 🔥 We will take a look asap 🚀 |
Beta Was this translation helpful? Give feedback.
-
|
This is not a bug, but indeed by design. Credentials, where the issuer is not trusted cannot be regarded as secure/trusted. It is the same with SSL certificates: unless they're issued by a trusted CA, browsers will likely reject them. Or you could compare this to using an OAuth2 access token that only grants permission to certain areas of a web-page, versus using an OAuth2 access token issued by a random third party. In the earlier case you'd simply expect to see only those parts you are allowed to see, whereas in the latter case you'd expect a HTTP 4xx. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Bug Report
Describe the Bug
✅ When requesting VCs for accessPolicy validation, the connector hides the corresponding offers where the policy evaluates to false (as expected).
❌ However, the connector returns an error (HTTP 401) if it receives VC from an untrusted issuer, even if only one offer in the catalog has an accessPolicy requiring this VC.
Expected Behavior
Catalog response is 200 with the visible offers (where accessPolicy evaluates to true).
Observed Behavior
Catalog response does not return any offers (HTTP 401) because one of the offers fails to validate a VC because it is from an untrusted issuer.
Steps to Reproduce
Setup:
Test:
e.g.
Context Information
Detailed Description
Connector A receives this error:
Connector B logs this error:
Adding an
edc.iam.trusted-issuer...for issuer X to connectorB's configuration resolves the issue. However, this is only a workaround.Possible Implementation
You already know the root cause of the erroneous state and how to fix it? Feel free to share your thoughts.
Beta Was this translation helpful? Give feedback.
All reactions