tecnickcom / nurago / 29316991257
100%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 194
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 100.0%. Remained the same
29316991257

push

github

nicolaasuni 
feat(redact)!: collapse the package-level API into Redactor, add an explicit opt-out, and split the package by concern

BREAKING CHANGE: the ten package-level redaction functions are gone. All redaction now runs through a *Redactor obtained from redact.Default() (shared, zero-config) or redact.New(opts...) (configured).

Migration is mechanical:
- redact.String(s) -> redact.Default().String(s)
- redact.Bytes(b) -> redact.Default().Bytes(b)
- redact.AppendTo(dst, src) -> redact.Default().AppendTo(dst, src)
- redact.Pooled(src, fn) -> redact.Default().Pooled(src, fn)
- redact.BytesToString(b) -> redact.Default().BytesToString(b)
- redact.HTTPData(s) -> redact.Default().String(s)
- redact.HTTPDataBytes(b) -> redact.Default().Bytes(b)
- redact.HTTPDataBytesInto(dst, src) -> redact.Default().AppendTo(dst, src)
- redact.HTTPDataBytesPooled(src, fn) -> redact.Default().Pooled(src, fn)
- redact.HTTPDataString(b) -> redact.Default().BytesToString(b)

The package carried three parallel surfaces over one engine: the HTTPData* compatibility aliases, the canonical package-level functions, and the Redactor methods. There is now one.

redact.Default() is retained deliberately, and is a shared instance rather than a convenience wrapper: a Redactor memoizes non-ASCII key classifications per instance, so sharing one keeps that cache bounded instead of allocating a fresh lazily-grown map per caller. httpclient, httpserver, and httpreverseproxy now fall back to redact.Default().BytesToString when WithRedactFn is omitted.

WithRedactFn keeps its func([]byte) string signature, so a Redactor method value plugs straight in and callers remain free to supply their own implementation.

Add redact.InsecureNoRedaction, a ready-made pass-through for the redact-function options, so switching redaction off no longer means hand-rolling an identity closure. It is named to be conspicuous in review, after crypto/tls.InsecureSkipVerify: redaction can now be lost only by naming this funct... (continued)

16803 of 16803 relevant lines covered (100.0%)

4707.74 hits per line

Jobs
ID Job ID Ran Files Coverage
1 29316991257.1 194
100.0
 GitHub Action Run
Source Files on build 29316991257