Skip to content

Conversation

@FarhanAnjum-opti
Copy link
Contributor

@FarhanAnjum-optiFarhanAnjum-opti commented Oct 8, 2025

Summary

Fixes concurrency bug in cmab service by introducing locking in order to ensure consistent result.

Test plan

Introduced unit tests

Issues

FSSDK-11901

Copy link

@prisma-cloud-devsecopsprisma-cloud-devsecopsbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prisma Cloud has found errors in this PR ⬇️

raju-opti
raju-opti previously requested changes Oct 8, 2025
Copy link
Contributor

@Mat001Mat001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now looks ok. Glad we are removing Py and pypy 3.8. It's getting out of security updates support.

Copy link
Contributor

@Mat001Mat001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@FarhanAnjum-optiFarhanAnjum-opti merged commit 116d23a into masterOct 15, 2025
35 of 36 checks passed
@FarhanAnjum-optiFarhanAnjum-opti deleted the farhan/FSSDK-11901-fix-concurrency-bug-in-cmab branch October 15, 2025 09:01
Sign up for freeto 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.

4 participants

@FarhanAnjum-opti@Mat001@raju-opti