Skip to content

Commit

Permalink
update redis uri to use redis://
Browse files Browse the repository at this point in the history
  • Loading branch information
cwaldren-ld committed Aug 9, 2024
1 parent e0af1b5 commit 6d38414
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ extern "C" {
*
* // Create the Redis source, passing in arguments for the URI, prefix, and
* // pointer to the result.
* if (!LDServerLazyLoadRedisSource_New("tcp://localhost:6379", "testprefix",
* if (!LDServerLazyLoadRedisSource_New("redis://localhost:6379", "testprefix",
* &result)) {
* // On failure, you may print the error message (result.error_message),
* // then exit or return.
Expand Down
4 changes: 3 additions & 1 deletion libs/server-sdk-redis-source/tests/redis_source_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,9 @@ using namespace launchdarkly::server_side;
class RedisTests : public ::testing::Test {
public:
explicit RedisTests()

Check warning on line 24 in libs/server-sdk-redis-source/tests/redis_source_test.cpp

View workflow job for this annotation

GitHub Actions / cpp-linter

/libs/server-sdk-redis-source/tests/redis_source_test.cpp:24:5 [cppcoreguidelines-pro-type-member-init]

constructor does not initialize these fields: source, client_
: uri_("tcp://localhost:6379"), prefix_("testprefix"), client_(uri_) {}
: uri_("redis://localhost:6379"),
prefix_("testprefix"),
client_(uri_) {}

void SetUp() override {
try {
Expand Down

0 comments on commit 6d38414

Please sign in to comment.