Ruby/Rails logs onboarding suggests removed enable_logs

Aperta Adatta ai principianti
  • Qualcosa non va?
#130,405 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rails, react, ruby, typescript
Ambito
documentation

Direzione di ricerca

Inizia da static/app/gettingStartedDocs/ruby-rails/onboarding.tsx, ruby/onboarding.tsx e ruby/logs.tsx, confrontando i loro snippet con il comportamento di sentry-ruby 7.x. Rimuovi le indicazioni su config.enable_logs = true e aggiorna la nota sulla versione e le indicazioni sul logging strutturato in modo che corrispondano alle modifiche di 7.0.0/7.1.0. Il lavoro è completo quando gli snippet di onboarding Ruby e Rails non mostrano più l'opzione rimossa e il testo sui log riflette la configurazione attuale di 7.x.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Bug Component: Onboarding

The Ruby and Rails onboarding snippets tell users to set config.enable_logs = true. sentry-ruby removed that option in 7.0.0, so on current SDK versions (7.x, latest 7.2.0) the snippet raises NoMethodError in Sentry.init. Reported by a customer setting up Rails with sentry-ruby 7.2.0.

Affected onboarding code (pinned to f182595):

SDK changes behind this:

  • 7.0.0, getsentry/sentry-ruby#3053: logs are enabled by default and enable_logs is removed. No compatibility shim remains
  • 7.1.0, getsentry/sentry-ruby#3102: Rails structured logging is off by default. Turn it on with config.rails.structured_logging.enabled = true

Workaround: drop the config.enable_logs = true line.

Open question: whether config.enabled_patches = [:logger] is still the recommended setup on 7.x.

cc @getsentry/value-discovery (CODEOWNERS for gettingStartedDocs) @sl0thentr0py (sentry-ruby)

via @danielsheccoury-sentry.

--

View Session in Sentry

Lingua principale
Python
Stelle
45.7k
Fork
5k
Merge medio
1g 1h
PR unite (30g)
642

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di getsentry/sentry

Tutte le issue di getsentry/sentry

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.