optimizely / agent / 17447035226

87%

push

github

[FSSDK-11793] Release/v4.2.1 (#441)

* fix: parse secure token from SDK key in notification handler

Handle SDK keys with secure tokens in format 'sdkKey:apiKey' by extracting
only the SDK key portion for notification processing.

* docs: improve Redis subscription documentation in config.yaml

Change comment from 'PSUBSCRIBE' to 'Subscribe/PSubscribe' to clarify
support for both Redis subscription patterns in notification sync.

* test: add comprehensive unit tests for secure token parsing

Add unit tests covering:
- Standard SDK keys without secure tokens
- Secure token format (sdkKey:apiKey) parsing
- Edge cases: multiple colons, empty parts, empty headers
- Integration test with notification event stream

Ensures secure token parsing logic has proper test coverage.

* fix: remove unused variables in secure token parsing tests

Remove unused 'conf' variables that were causing linting errors
in CI checks for the new secure token parsing unit tests.

* fix: remove trailing spaces in secure token parsing tests

Clean up trailing whitespace that was causing formatting issues
in CI checks for golangci-lint.

* fix: update Go version from 1.24 to 1.23 (1.24 doesn't exist yet)

* chore: add CHANGELOG entry for v4.2.1 and fix test formatting

* fix: update Alpine version from 3.21 to 3.20 for Go 1.23 compatibility

* fix: prevent test hanging by using context timeout instead of closing channel

The previous test was closing the event channel immediately, which caused
the notification handler to hang in an infinite loop reading zero values.
Fix by using a context timeout to properly terminate the test.

* format

4 of 4 new or added lines in 1 file covered. (100.0%)

2874 of 3335 relevant lines covered (86.18%)

4441.81 hits per line