Skip to content

docs/observability: Remove stale log15 examples - #1884

Open
marcleblanc2 wants to merge 2 commits into
mainfrom
fix-stale-log-level-docs
Open

docs/observability: Remove stale log15 examples#1884
marcleblanc2 wants to merge 2 commits into
mainfrom
fix-stale-log-level-docs

Conversation

@marcleblanc2

@marcleblanc2 marcleblanc2 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Sourcegraph replaced `inconshreveable/log15` with sourcegraph/log years ago, so nothing prints `lvl=eror` anymore. Three docs still show it.

  • `self-hosted/observability/logs.mdx`: list the canonical `SRC_LOG_LEVEL` values (`debug`, `info`, `warn`, `error`, `none`) per levels.go, and note that unknown values fall back to `warn`. The legacy `dbug`/`eror`/`crit` spellings are still accepted by the code but are no longer documented.
  • `self-hosted/observability/troubleshooting.mdx`: drop the 2020 log15 sample line. The `syncer.sync.store.upsert-repos` error string no longer exists either; keep the scenario and describe the `driver: bad connection` symptom in prose.
  • `admin/auth/index.mdx`: the `Invalid Authorization header` log15 error is gone. Per `cmd/frontend/internal/auth/accesstoken/auth.go`, an unrecognized `Authorization` scheme is now ignored and recorded as an `auth.accessToken` / `failed` audit log entry with `reason: "invalid Authorization header"`.

Amp threads

SRC_LOG_LEVEL: list the canonical debug/info/warn/error/none values and
note dbug/eror/crit as legacy aliases. Drop the 2020 log15 sample line
from the worker troubleshooting scenario. Describe the current audit-log
behavior for unrecognized Authorization headers instead of the removed
'Invalid Authorization header' log15 error.

Amp-Thread-ID: https://ampcode.com/threads/T-01a082f4-e864-769b-8269-46847abcd228
Co-authored-by: Amp <amp@ampcode.com>
@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sourcegraph-docs Ready Ready Preview Sep 9, 2026 3:32am UTC

Request Review

@marcleblanc2 marcleblanc2 changed the title Remove stale log15 lvl=eror samples from docs Remove stale log15 examples from docs Sep 9, 2026
@marcleblanc2 marcleblanc2 changed the title Remove stale log15 examples from docs docs/observability: Remove stale log15 examples Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant