romshark / datapages / 39652905704

53%

Build Type

push

github

Committed by romshark

Commit Message

fix: Recover panics while reading request inputs

- fix: Register the recover of every generated handler as its first
  statement.
  Before: a panic while a handler reads its session, path, query or
  signals, as in UnmarshalText, UnmarshalJSON or a sessions.Manager,
  reaches net/http, which closes the connection without a response.
  A page load gets no 500 page, and a Datastar request no
  RecoverError.
  After: such a panic takes the path of a panic in the handler. The
  stack is logged. A page load gets PageError500 or status 500, and a
  Datastar request gets RecoverError, which receives a
  datapages.PanicError, or status 500.
- fix: Allocate a pointer path or query field, such as *big.Int,
  before its UnmarshalText runs.
  Before: UnmarshalText runs on a nil pointer, which panics on every
  request that carries the parameter.
  After: the field points to the parsed value. A query field whose
  parameter is absent stays nil.

Coverage Stats

472 of 1350 new or added lines in 48 files covered. (34.96%)

16 existing lines in 2 files now uncovered.

33069 of 63115 relevant lines covered (52.39%)

4365.05 hits per line