Remove recover from Redis config store (#2124)
Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com> Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com> Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
This commit is contained in:
parent
d3ec1ff954
commit
b566232f67
|
@ -312,11 +312,6 @@ func (r *ConfigurationStore) doSubscribe(ctx context.Context, req *configuration
|
|||
}
|
||||
|
||||
func (r *ConfigurationStore) handleSubscribedChange(ctx context.Context, req *configuration.SubscribeRequest, handler configuration.UpdateHandler, msg *redis.Message, id string) {
|
||||
defer func() {
|
||||
if err := recover(); err != nil {
|
||||
r.logger.Errorf("panic in handleSubscribedChange()method and recovered: %s", err)
|
||||
}
|
||||
}()
|
||||
targetKey, err := internal.ParseRedisKeyFromEvent(msg.Channel)
|
||||
if err != nil {
|
||||
r.logger.Errorf("parse redis key failed: %s", err)
|
||||
|
|
Loading…
Reference in New Issue