Project

General

Profile

Actions

Improvements #14479

closed

Uncompressed json response with application/json

Added by Swan Bosc 4 months ago. Updated 24 days ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
04/16/2026
Due date:
% Done:

0%

Estimated time:

Description

Hi,
The logic to decide whether a "Reply" in Reply.c is compressed is based on the mime type.
It is quite limiting as many response might not set it properly and many mime types exists that are note supported by this "or" (see https://www.iana.org/assignments/media-types/media-types.xhtml)
For example "application/json" isn't compressed.

What I would do (actually did in our project that uses wt) is to compress everything. This is what google does, because the overhead is quite okay and it is better to compress uncompressible objects than to not compress compressible ones.

The other option is to maintain a set of uncompressible mime types and to add a mime type to the set once it hasn't been compressed successfully.

Regards,

Actions #1

Updated by Romain Mardulyn 4 months ago

  • Tracker changed from Bug to Improvements
  • Target version set to 4.14.0
Actions #2

Updated by Romain Mardulyn 3 months ago

  • Status changed from New to InProgress
  • Assignee set to Romain Mardulyn
Actions #3

Updated by Romain Mardulyn 3 months ago

  • Status changed from InProgress to Review
  • Assignee deleted (Romain Mardulyn)
Actions #4

Updated by Romain Mardulyn 3 months ago

  • Tracker changed from Improvements to Feature
Actions #5

Updated by Romain Mardulyn 3 months ago

  • Tracker changed from Feature to Improvements
Actions #6

Updated by emil de keyser about 1 month ago

  • Assignee set to emil de keyser
Actions #7

Updated by emil de keyser about 1 month ago

  • Status changed from Review to Resolved
  • Assignee changed from emil de keyser to Romain Mardulyn
Actions #8

Updated by Romain Mardulyn about 1 month ago

  • Status changed from Resolved to Implemented @Emweb
Actions #9

Updated by Romain Mardulyn 24 days ago

  • Status changed from Implemented @Emweb to Closed
Actions

Also available in: Atom PDF