Skip to content

Commit

Permalink
Fix APPSIGNAL_ACTIVEJOB_REPORT_ERRORS var naming (#1103)
Browse files Browse the repository at this point in the history
The config option is `activejob_report_errors`, but the env var had an
underscore between "ACTIVE" and "JOB".

We never documented the env var, so we can hopefully stealth fix this
without requiring a backwards compatibility change.

After this name change, let's document the env var.

[skip changeset]
  • Loading branch information
tombruijn authored Jun 21, 2024
1 parent 661b8e0 commit 75b1832
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/appsignal/config.rb
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class Config

ENV_TO_KEY_MAPPING = {
"APPSIGNAL_ACTIVE" => :active,
"APPSIGNAL_ACTIVE_JOB_REPORT_ERRORS" => :activejob_report_errors,
"APPSIGNAL_ACTIVEJOB_REPORT_ERRORS" => :activejob_report_errors,
"APPSIGNAL_APP_NAME" => :name,
"APPSIGNAL_BIND_ADDRESS" => :bind_address,
"APPSIGNAL_CA_FILE_PATH" => :ca_file_path,
Expand Down Expand Up @@ -117,7 +117,7 @@ class Config
}.freeze
# @api private
ENV_STRING_KEYS = %w[
APPSIGNAL_ACTIVE_JOB_REPORT_ERRORS
APPSIGNAL_ACTIVEJOB_REPORT_ERRORS
APPSIGNAL_APP_NAME
APPSIGNAL_BIND_ADDRESS
APPSIGNAL_CA_FILE_PATH
Expand Down

0 comments on commit 75b1832

Please sign in to comment.