Skip to content

Commit f75c03f

Browse files
committed
RR: keep q parameter visible in HttpHeaders.getAcceptableMediaTypes; still preserve order on ties
1 parent a3aab72 commit f75c03f

File tree

1 file changed

+2
-16
lines changed
  • independent-projects/resteasy-reactive/common/runtime/src/main/java/org/jboss/resteasy/reactive/common/headers

1 file changed

+2
-16
lines changed

independent-projects/resteasy-reactive/common/runtime/src/main/java/org/jboss/resteasy/reactive/common/headers/HeaderUtil.java

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -288,23 +288,9 @@ public static List<MediaType> getAcceptableMediaTypes(MultivaluedMap<String, ? e
288288
}
289289
}
290290
// Sort by weight (q) while preserving header-provided order on total ties
291+
// Keep the q parameter present for visibility (TCK expects it), but do not
292+
// rely on it downstream for matching; matching logic uses MediaTypeHelper.
291293
MediaTypeHelper.sortByWeight(list);
292-
293-
// The returned acceptable media types must not include the quality factor parameter ("q").
294-
// Strip any present "q" from the parameters while keeping other parameters (e.g., charset).
295-
for (int i = 0; i < list.size(); i++) {
296-
MediaType mt = list.get(i);
297-
Map<String, String> params = mt.getParameters();
298-
if (params != null && params.containsKey("q")) {
299-
if (params.size() == 1) {
300-
list.set(i, new MediaType(mt.getType(), mt.getSubtype()));
301-
} else {
302-
Map<String, String> copy = new java.util.LinkedHashMap<>(params);
303-
copy.remove("q");
304-
list.set(i, new MediaType(mt.getType(), mt.getSubtype(), copy));
305-
}
306-
}
307-
}
308294
return list;
309295
}
310296

0 commit comments

Comments
 (0)