tecnickcom / gogen / 28753903637

100%

push

github

feat(traceid): export Valid, replace regex with allocation-free scan, and harden nil handling

Replace the per-request regexp validator with an allocation-free byte scan (Valid/isIDByte); it is ~20x faster (~455ns to ~17ns/op, 0 allocs) and removes the regexp import and the package-level compiled-regex global.

Export Valid and MaxIDLen so callers can validate an untrusted trace ID before storing or logging it, and document that the context helpers (NewContext/ForceContext/FromContext) store IDs verbatim without validation — closing a log-injection gap at the API boundary as defense-in-depth.

Guard SetHTTPRequestHeaderFromContext and FromHTTPRequestHeader against a nil *http.Request, and lazily initialize a nil Header map, so they return gracefully instead of panicking (per the no-panic-in-library policy).

Clarify that SetHTTPRequestHeaderFromContext leaves any pre-existing header value unchanged (not "unset") when the resolved ID is empty or invalid.

Add ForceContext to the package overview and de-duplicate the validation-rule description so it lives in one place (Valid).

Add TestValid boundary coverage, nil-input tests, benchmarks, and an ExampleValid; coverage stays at 100%.

README: describe traceid as trace ID propagation and context management, since the package does not generate IDs.

27 of 27 new or added lines in 1 file covered. (100.0%)

12966 of 12970 relevant lines covered (99.97%)

100501.23 hits per line