cockroachdb / cockroachdb/errors

Stop relying on `pkg/errors` to handle stack traces

Ouverte
#70 7 commentaires 7 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Go
Étoiles
2.5k
Forks
74
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Today the stack trace formatting code delegates to `pkg/errors/stack.go`.

However, that code uses `runtime.FuncForPC` and emits file paths in the stack trace derived from the raw file name of the source file.

This method is outdated, as the on-disk file name of a given package's source may be different from the package path (due to go mod versioning, reproducible build sandboxes, etc).

The "modern" equivalent is `runtime.CallersFrame`, which populates `runtime.Frame` structs with package path-qualified function names.

We want to use that instead.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.