feat(rails): forward Rails.logger to PostHog Logs via OpenTelemetry by johnnagro · Pull Request #167 · PostHog/posthog-ruby

feat(rails): forward Rails.logger to PostHog Logs via OpenTelemetry

johnnagro

committed

fix(rails): drop incorrect service.version OTel resource attribute

johnnagro

committed

test(rails): cover all severity mappings in appender spec

johnnagro

committed

refactor(rails): capture init options for logs instead of exposing client readers

johnnagro

committed

docs(rails): document Appender's lock-free thread-safety design

johnnagro

committed

fix(rails): match self-log prefix with start_with? to avoid over-suppression

johnnagro

committed

feat(rails): cap PostHog Logs forwarding at a configurable records-per-minute rate

johnnagro

committed

feat(rails): add logs_before_send callback to scrub or drop PostHog Logs records

johnnagro

committed

Update posthog-rails/lib/posthog/rails/logs/appender.rb

johnnagro

and

turnipdabeets

committed

fix: fallback to 'unknown_service' rather than 'rails'

johnnagro

committed

fix(rails): emit progname as logger.name to match OTel logger-name convention

johnnagro

committed

docs(rails): note automatic trace_id/span_id correlation for apps using OTel tracing

johnnagro

committed

refactor(rails): expose a single :severity enum in the logs_before_send record hash

johnnagro

committed

fix(rails): run logs_before_send before the rate cap so dropped records don't consume budget

johnnagro

committed

fix(rails): use OTel semconv names (url.full, http.request.method, url.path) for log request attributes

johnnagro

committed

test(rails): cover body UTF-8 normalization and before_send string safety

johnnagro

committed

fix(rails): warn when logs_before_send returns a non-Hash value instead of silently dropping

johnnagro

committed

fix(rails): coerce numeric strings for logs rate cap instead of silently disabling it

johnnagro

committed

docs(rails): recommend require: false for the optional OpenTelemetry gems

johnnagro

committed

fix(rails): warn when logs are enabled but PostHog.init never ran

johnnagro

committed

fix(rails): bound the at_exit logs flush with a 2s timeout so it can't starve the events flush

johnnagro

committed

refactor(rails): rename forward_rails_logger to logs_forward_rails_logger for prefix consistency

johnnagro

committed

Remove logs_resource_attributes config; resource attrs are now fixed service metadata

johnnagro

committed

Remove unused Setup.force_flush; shutdown! already flushes buffered records

johnnagro

committed

Warn once on invalid logs_level instead of silently falling back to the Rails logger level

johnnagro

committed

Keep forwarding threshold out of Logger#level so BroadcastLogger predicates and level= are unaffected; document silence/tagged gaps

johnnagro

committed

Guard Appender#add against re-entrant logging and warn once when emit fails persistently

johnnagro

committed

Add fork-safety spec pinning BatchLogRecordProcessor's post-fork restart through the logs pipeline

johnnagro

committed

feat(rails): add changeset and fork-safety spec for PostHog Logs forwarding

johnnagro

committed

Remove unused Severity.for; name_for and for_name cover the split emit path

johnnagro

committed

Merge remote-tracking branch 'origin/main' into feat/posthog-rails-otel-logs

johnnagro

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(rails): forward Rails.logger to PostHog Logs via OpenTelemetry#167

feat(rails): forward Rails.logger to PostHog Logs via OpenTelemetry#167
johnnagro wants to merge 31 commits into
PostHog:mainfrom
noreastergroup:feat/posthog-rails-otel-logs

Commits on Jun 12, 2026

feat(rails): forward Rails.logger to PostHog Logs via OpenTelemetry

fix(rails): drop incorrect service.version OTel resource attribute

test(rails): cover all severity mappings in appender spec

refactor(rails): capture init options for logs instead of exposing client readers

docs(rails): document Appender's lock-free thread-safety design

fix(rails): match self-log prefix with start_with? to avoid over-suppression

feat(rails): cap PostHog Logs forwarding at a configurable records-per-minute rate

feat(rails): add logs_before_send callback to scrub or drop PostHog Logs records

Update posthog-rails/lib/posthog/rails/logs/appender.rb

fix: fallback to 'unknown_service' rather than 'rails'

fix(rails): emit progname as logger.name to match OTel logger-name convention

docs(rails): note automatic trace_id/span_id correlation for apps using OTel tracing

refactor(rails): expose a single :severity enum in the logs_before_send record hash

fix(rails): run logs_before_send before the rate cap so dropped records don't consume budget

fix(rails): use OTel semconv names (url.full, http.request.method, url.path) for log request attributes

test(rails): cover body UTF-8 normalization and before_send string safety

fix(rails): warn when logs_before_send returns a non-Hash value instead of silently dropping

fix(rails): coerce numeric strings for logs rate cap instead of silently disabling it

docs(rails): recommend require: false for the optional OpenTelemetry gems

fix(rails): warn when logs are enabled but PostHog.init never ran

fix(rails): bound the at_exit logs flush with a 2s timeout so it can't starve the events flush

refactor(rails): rename forward_rails_logger to logs_forward_rails_logger for prefix consistency

Remove logs_resource_attributes config; resource attrs are now fixed service metadata

Remove unused Setup.force_flush; shutdown! already flushes buffered records

Warn once on invalid logs_level instead of silently falling back to the Rails logger level

Keep forwarding threshold out of Logger#level so BroadcastLogger predicates and level= are unaffected; document silence/tagged gaps

Guard Appender#add against re-entrant logging and warn once when emit fails persistently

Add fork-safety spec pinning BatchLogRecordProcessor's post-fork restart through the logs pipeline

feat(rails): add changeset and fork-safety spec for PostHog Logs forwarding

Remove unused Severity.for; name_for and for_name cover the split emit path

Merge remote-tracking branch 'origin/main' into feat/posthog-rails-otel-logs