Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configuration: support allowlists #2546

Merged
merged 7 commits into from
Apr 12, 2024
Merged

Configuration: support allowlists #2546

merged 7 commits into from
Apr 12, 2024

Commits on Apr 10, 2024

  1. Configuration: support allowlists

    Configuration hashes defined by `default_source.rb` now support an
    additional `:allowlist` key set equal to an array of valid values.
    
    If an invalid value is specified for a parameter that has an allowlist,
    a warning will be logged that contains the allowlist and the default
    value for the parameter will be used.
    fallwith committed Apr 10, 2024
    Configuration menu
    Copy the full SHA
    cda8672 View commit details
    Browse the repository at this point in the history

Commits on Apr 11, 2024

  1. Configuration menu
    Copy the full SHA
    3604b29 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    2a2dac8 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2024

  1. remove unnecessary :: prefix

    ::NewRelic -> NewRelic
    fallwith committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    642ca83 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    cae2356 View commit details
    Browse the repository at this point in the history
  3. Config allowlists: let config handle validation

    Remove the class specific config validation that is now redundant with
    the config's own allowlist validation
    fallwith committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    15cf26b View commit details
    Browse the repository at this point in the history
  4. remove 8T test related to config allowlist

    Now that the config handles the allowlist validation, remove the test
    expecting the 8T class to do so
    fallwith committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    8ff155a View commit details
    Browse the repository at this point in the history