Skip to content

Conversation

@SimonCropp
Copy link
Contributor

fixes #4893

since there is a continue in the loop. it is not safe to pre-allocate the array size

@SimonCropp SimonCropp mentioned this pull request Mar 20, 2025
@bording bording changed the base branch from master to fix-saga-audit-forwarding March 20, 2025 22:42
@bording bording merged commit 19fa89c into Particular:fix-saga-audit-forwarding Mar 20, 2025
3 of 27 checks passed
bording added a commit that referenced this pull request Mar 20, 2025
* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
bording added a commit that referenced this pull request Mar 20, 2025
* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
bording added a commit that referenced this pull request Mar 20, 2025
* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
bording added a commit that referenced this pull request Mar 21, 2025
* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
bording added a commit that referenced this pull request Mar 21, 2025
* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
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.

ServiceControl Audit Ingestion throwing NREs with audit forwarding enabled

2 participants