Skip to content

[http] by default allow decode of post object#22487

Merged
linev merged 1 commit into
root-project:masterfrom
linev:http_post_exec_on
Jun 8, 2026
Merged

[http] by default allow decode of post object#22487
linev merged 1 commit into
root-project:masterfrom
linev:http_post_exec_on

Conversation

@linev

@linev linev commented Jun 5, 2026

Copy link
Copy Markdown
Member

Option can be used to prevent complex object decoding when exe.json request is executed.

It was default behaviour before.

Follow up for #22186

Option can be used to prevent complex object decoding when
exe.json request is executed.

It was default behaviour before.
@linev linev requested review from bellenot and dpiparo June 5, 2026 10:38
@linev linev self-assigned this Jun 5, 2026
@dpiparo dpiparo requested a review from jblomer June 5, 2026 11:41
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 11h 23m 28s ⏱️
 3 854 tests  3 854 ✅ 0 💤 0 ❌
76 138 runs  76 138 ✅ 0 💤 0 ❌

Results for commit 429e998.

@linev linev added this to the 6.40.02 milestone Jun 5, 2026
@jblomer

jblomer commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Could you remind me: why do we need to change (back) the default? Did anything break with the changed behaviour?

@linev

linev commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Could you remind me: why do we need to change (back) the default? Did anything break with the changed behaviour?

This is new flag which was introduced with #22186
But if changes default behavior of THttpServer.
Therefore I want to switch this option true.
As we discuss - I/O for arbitrary class is not an issue when host system was not compromised before

@linev linev merged commit d4d1955 into root-project:master Jun 8, 2026
36 of 37 checks passed
@linev

linev commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

/backport to 6.40, 6.38, 6.36, 6.32, 6.30, 6.28, 6.26

@linev linev deleted the http_post_exec_on branch June 8, 2026 13:01
@root-project-bot

Copy link
Copy Markdown

Preparing to backport PR #22487 to branches 6.40, 6.38, 6.36, 6.32, 6.30, 6.28 and 6.26 requested by linev

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants