Revert "Handle nil logger again"

This reverts commit 66ac1f58ce.
This commit is contained in:
Alex Peters
2022-03-02 14:15:11 +01:00
parent 66ac1f58ce
commit 654fcec122

View File

@@ -134,7 +134,7 @@ func (d MessageDispatcher) DispatchSubmessages(ctx sdk.Context, contractAddr sdk
}
} else {
// Issue #759 - we don't return error string for worries of non-determinism
logRedactError(ctx, err)
moduleLogger(ctx).Info("Redacting submessage error", "cause", err)
result = wasmvmtypes.SubcallResult{
Err: redactError(err),
}
@@ -159,13 +159,6 @@ func (d MessageDispatcher) DispatchSubmessages(ctx sdk.Context, contractAddr sdk
return rsp, nil
}
func logRedactError(ctx sdk.Context, err error) {
logger := moduleLogger(ctx)
if logger != nil {
logger.Info("Redacting submessage error", "cause", err)
}
}
func redactError(err error) string {
// FIXME: do we want to hardcode some constant string mappings here as well?
// Or better document them? (SDK error string may change on a patch release to fix wording)