Skip to content

[1.5.x] [bug] OptionalField race condition (#12627) - #12629

Merged
SchrodingersGat merged 1 commit into
1.5.xfrom
backport/1.5.x/pr-12627
Aug 12, 2026
Merged

[1.5.x] [bug] OptionalField race condition (#12627)#12629
SchrodingersGat merged 1 commit into
1.5.xfrom
backport/1.5.x/pr-12627

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from master to 1.5.x:

Questions ?

Please refer to the Backport tool documentation

* [bug] OptionalField race condition

Fixes subtle bug where OptionalField entries can be  silenty dropped from an API request due to concurrent requests / race conditions

* Additional unit tests

* Additional guard in metadata.py

* include extra kwargs

* Adjust import/exporting options

* Fix attribute sharing across class instances

(cherry picked from commit 2b3283d)
@SchrodingersGat SchrodingersGat added bug Identifies a bug which needs to be addressed api Relates to the API labels Aug 12, 2026
@SchrodingersGat SchrodingersGat added this to the 1.4.5 milestone Aug 12, 2026
@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.05882% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.78%. Comparing base (be566f4) to head (6957a1a).

Additional details and impacted files
@@           Coverage Diff           @@
##            1.5.x   #12629   +/-   ##
=======================================
  Coverage   86.78%   86.78%           
=======================================
  Files        1448     1448           
  Lines       96799    96849   +50     
  Branches    11267    11267           
=======================================
+ Hits        84003    84055   +52     
+ Misses      12732    12730    -2     
  Partials       64       64           
Flag Coverage Δ
backend 90.77% <97.05%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 92.15% <100.00%> (+<0.01%) ⬆️
Backend General 93.56% <100.00%> (+0.02%) ⬆️
Frontend 79.72% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SchrodingersGat
SchrodingersGat merged commit 9af4452 into 1.5.x Aug 12, 2026
50 of 51 checks passed
@SchrodingersGat
SchrodingersGat deleted the backport/1.5.x/pr-12627 branch August 12, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Relates to the API bug Identifies a bug which needs to be addressed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant