Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix/remove kwargs injection#1582

Merged
collerek merged 2 commits into
masterormar-orm/ormar:masterfrom
fix/remove-kwargs-injectionormar-orm/ormar:fix/remove-kwargs-injectionCopy head branch name to clipboard
Mar 19, 2026
Merged

Fix/remove kwargs injection#1582
collerek merged 2 commits into
masterormar-orm/ormar:masterfrom
fix/remove-kwargs-injectionormar-orm/ormar:fix/remove-kwargs-injectionCopy head branch name to clipboard

Conversation

@collerek
Copy link
Copy Markdown
Collaborator

No description provided.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 19, 2026

Merging this PR will degrade performance by 29.52%

⚡ 1 improved benchmark
❌ 1 (👁 1) regressed benchmark
✅ 82 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
👁 WallTime test_creating_individually_with_related_models[10] 56.4 ms 80 ms -29.52%
WallTime test_updating_models_individually[20] 147.3 ms 132.8 ms +10.93%

Comparing fix/remove-kwargs-injection (3620c92) with master (8e144d0)

Open in CodSpeed

@collerek collerek merged commit 7f22aa2 into master Mar 19, 2026
18 checks passed
@collerek collerek deleted the fix/remove-kwargs-injection branch March 19, 2026 14:55
@michaelm-openai
Copy link
Copy Markdown

Hi @collerek, would it be possible to backport this and #1557 to 0.20.*? Since versions >=0.21.0 started requiring sqlalchemy 2.0, backporting would provide a path to quickly mitigate the vulnerabilities without the risk of having to rush a major sqlalchemy upgrade.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.