Project

General

Profile

Actions

Improvements #14479

closed
SB RM

Uncompressed json response with application/json

Improvements #14479: Uncompressed json response with application/json

Added by Swan Bosc 5 months ago. Updated about 2 months 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,

RM Updated by Romain Mardulyn 5 months ago Actions #1

  • Tracker changed from Bug to Improvements
  • Target version set to 4.14.0

RM Updated by Romain Mardulyn 4 months ago Actions #2

  • Status changed from New to InProgress
  • Assignee set to Romain Mardulyn

RM Updated by Romain Mardulyn 4 months ago Actions #3

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

RM Updated by Romain Mardulyn 4 months ago Actions #4

  • Tracker changed from Improvements to Feature

RM Updated by Romain Mardulyn 4 months ago Actions #5

  • Tracker changed from Feature to Improvements

ED Updated by emil de keyser 2 months ago Actions #6

  • Assignee set to emil de keyser

ED Updated by emil de keyser 2 months ago Actions #7

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

RM Updated by Romain Mardulyn 2 months ago Actions #8

  • Status changed from Resolved to Implemented @Emweb

RM Updated by Romain Mardulyn about 2 months ago Actions #9

  • Status changed from Implemented @Emweb to Closed
Actions

Also available in: PDF Atom