Skip to content

feat: introduce object to represent an HTTP request problem details - #799

Open
romm wants to merge 1 commit into
CuyZ:masterfrom
romm:feat/http-request-problem-details
Open

feat: introduce object to represent an HTTP request problem details#799
romm wants to merge 1 commit into
CuyZ:masterfrom
romm:feat/http-request-problem-details

Conversation

@romm

@romm romm commented Mar 26, 2026

Copy link
Copy Markdown
Member

No description provided.

@boesing

boesing commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

I wonder why valinor needs these kind of details.
Yes, this is an RFC standard, yet it feels odd that this type of detail is held within valinor.

I have seen that you are using it in the mezzio package and it might be useful in symfony bundle, yet this is not related to the core of valinor (i.e. normalization/deserialization).

I am not against this, it just feels like something which should reside in the respective package(s).


Oh and btw., the "errors" are actually not RFC. Only status, type, detail and title are.
Just wondering if this follows some framework specific details. For example in symfony, the errors are mapped differently than in mezzio.

@romm

romm commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Hi @boesing, thanks for your interest!

The work on this is to ease the use of RFC-7807 problems when using Valinor for HTTP request mapping (you may have missed this new feature from not so long ago).

HTTP request mapping is one of the usages that makes most sense with Valinor, that's why I wanted it provided out of the box by the library. Problem details is a logical follow-up. 😊

Just wondering if this follows some framework specific details. For example in symfony, the errors are mapped differently than in mezzio.

Actually this is more RFC-compliant than any-framework compliant, and I'm pretty sure we can find packages that do this kind of job for well-known frameworks. When this PR is merged, I will provide an implementation for Symfony and Mezzio bundles/packages to automatically make us of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants