CrowdStrike / csproto

CrowdStrike's Protocol Buffers library
MIT License
122 stars 13 forks source link

fix: RangeExtensions doesn't enumerate all items for Google V2 #112

Closed dylan-bourque closed 1 year ago

dylan-bourque commented 1 year ago

The Google V2 RangeExtensions API takes a callback that returns bool instead of error and we were returning err != nil, which returns false if there was no error and exits the enumeration.

This PR fixes the logic and adds unit tests for all three supported runtimes.