Might be a tiny bug in “p_redis.go”.

[issuefiler]

go-guerrilla/backends/p_redis.go

Line 100 in 51f7dda

redisErr = redisClient.redisConnection(config.RedisInterface)

go-guerrilla/backends/p_redis.go

Lines 106 to 111 in 51f7dda

	_, doErr := redisClient.conn.Do("SETEX", hash, config.RedisExpireSeconds, stringer)
	if doErr != nil {
	Log().WithError(doErr).Warn("Error while SETEX to redis")
	result := NewResult(response.Canned.FailBackendTransaction)
	return result, redisErr
	}

Shouldn’t it return result, doErr?

[flashmob]

Here we don't really care about the result, because were using SETEX (saving data)

[issuefiler]

go-guerrilla/backends/p_redis.go

Lines 100 to 111 in 51f7dda

	redisErr = redisClient.redisConnection(config.RedisInterface)
	if redisErr != nil {
	Log().WithError(redisErr).Warn("Error while connecting to redis")
	result := NewResult(response.Canned.FailBackendTransaction)
	return result, redisErr
	}
	_, doErr := redisClient.conn.Do("SETEX", hash, config.RedisExpireSeconds, stringer)
	if doErr != nil {
	Log().WithError(doErr).Warn("Error while SETEX to redis")
	result := NewResult(response.Canned.FailBackendTransaction)
	return result, redisErr
	}

I mean, the line number 110: return result, redisErr. Does it really mean to return redisErr that is a nil?

[flashmob]

Indeed, line 110 should return doErr.

Todo: Fix

[flashmob]

fixed in #202 https://github.com/flashmob/go-guerrilla/pull/202/files#diff-a1c0f6ddb8710df8182db68e5085731dR110