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

Default proxy config error management needs a review #373

Open
Zicchio opened this issue Feb 28, 2025 · 1 comment
Open

Default proxy config error management needs a review #373

Zicchio opened this issue Feb 28, 2025 · 1 comment
Assignees
Labels
enhancement Something improving existing features standardization
Milestone

Comments

@Zicchio
Copy link
Collaborator

Zicchio commented Feb 28, 2025

The current proxy configuration is designed so that when un uncatght exception is thrown at some endpoint (response endpoint or request endpoint), then a generic error page is returned.

This page is intended to be read by a user but it not correct for http requests that starts from a wallet, such as an authorization response in direct_post mode. In those cases, we should always provide error responses compliant with openid4vp and the italian profile for openid4vp+sd-jwt

@peppelinux
Copy link
Member

this seems related to iam-proxy-italia

at the same time this issue aims to clarify that for the wallet flow we need pure json responses according to openid4vp and not an http page

we have already improved the error handling using only json responses, this issue will be resolved once the review will enrich this thread with the evidences about the missing error handling

@peppelinux peppelinux added this to the 1.1.0 milestone Mar 6, 2025
@peppelinux peppelinux added enhancement Something improving existing features standardization labels Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Something improving existing features standardization
Projects
Status: Todo
Development

No branches or pull requests

2 participants