search

launchdarkly / cpp-sdks

C++ Client/Server SDKs
Other
5 stars 2 forks source link

feat: allow LDStatus to carry an arbitrary string message #342

Closed cwaldren-ld closed 8 months ago

cwaldren-ld commented 9 months ago

Our C bindings return LDStatus, which wraps an Error enum. This is an opaque type so that we could modify how the error was represented.

That time has come, and I've modified the underlying Error type to be a variant<ErrorCode, string>. This supports reporting arbitrary string messages. The immediate use case is reporting errors from the Redis integration, which doesn't use status codes.

shortcut-integration[bot] commented 9 months ago

This pull request has been linked to Shortcut Story #227034: Allow LDStatus to report arbitrary string errors.