[PR #15] [MERGED] Do not return highest priority format immediately when catch-all is dete... #26

New Issue

admin · 2026-01-24T11:38:02+01:00

admin commented

2026-01-24 11:38:02 +01:00

📋 Pull Request Information

Original PR: https://github.com/FriendsOfSymfony/FOSRest/pull/15
Author: @teohhanhui
Created: 7/9/2013
Status: ✅ Merged
Merged: 7/24/2013
Merged by: @lsmith77

Base: master ← Head: patch-1

📝 Commits (2)

89306dd Do not return highest priority format immediately when catch-all is detected
cf699ff Update FormatNegotiator.php

📊 Changes

1 file changed (+7 additions, -1 deletions)

View changed files

📝 Util/FormatNegotiator.php (+7 -1)

📄 Description

...cted

Related to issue 372 of FOSRestBundle: https://github.com/FriendsOfSymfony/FOSRestBundle/issues/372

From the HTTP/1.1 spec (http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html):

Media ranges can be overridden by more specific media ranges or specific media types. If more than one media range applies to a given type, the most specific reference has precedence. For example,

   Accept: text/*, text/html, text/html;level=1, */*

have the following precedence:

   1) text/html;level=1
   2) text/html
   3) text/*
   4) */*

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/FriendsOfSymfony/FOSRest/pull/15 **Author:** [@teohhanhui](https://github.com/teohhanhui) **Created:** 7/9/2013 **Status:** ✅ Merged **Merged:** 7/24/2013 **Merged by:** [@lsmith77](https://github.com/lsmith77) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (2) - [`89306dd`](https://github.com/FriendsOfSymfony/FOSRest/commit/89306dd9721db96c7ae58f9276eb814732f5271d) Do not return highest priority format immediately when catch-all is detected - [`cf699ff`](https://github.com/FriendsOfSymfony/FOSRest/commit/cf699ffe6573a12dc7c4b19768a4ada4ad551842) Update FormatNegotiator.php ### 📊 Changes **1 file changed** (+7 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Util/FormatNegotiator.php` (+7 -1) </details> ### 📄 Description ...cted Related to issue 372 of FOSRestBundle: https://github.com/FriendsOfSymfony/FOSRestBundle/issues/372 From the HTTP/1.1 spec (http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html): Media ranges can be overridden by more specific media ranges or specific media types. If more than one media range applies to a given type, the most specific reference has precedence. For example, ``` Accept: text/*, text/html, text/html;level=1, */* ``` have the following precedence: ``` 1) text/html;level=1 2) text/html 3) text/* 4) */* ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

admin added the pull-request label 2026-01-24 11:38:02 +01:00

admin closed this issue

2026-01-24 11:38:02 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FriendsOfSymfony/FOSRest#26