Skip to content
  • Yutaka Kamei's avatar
    0b7f37fb
    Initialize `ActionController::Parameters` with `@logging_context` · 0b7f37fb
    Yutaka Kamei authored
    `params` contains `@logging_context` in its instance to notify
    unpermitted parameters including the context through Rails
    Instrumentation API. However, the logging context disappeared when
    `params` is updated with some methods, such as `require`, `slice`,
    `merge`, etc, so the subscriber of `unpermitted_parameters` could not
    get the information.
    
    This patch tries to initialize `Parameters` with `@logging_context`
    where it makes sense to pass the information. The following methods will
    be affected with this patch:
    
    * `require`
    * `deep_dup`
    * `slice`
    * `except`
    * `extract!`
    * `transform_values`
    * `transform_keys`
    * `deep_transform_keys`
    * `select`
    * `reject`
    * `compact`
    * `merge`
    * `reverse_merge`
    0b7f37fb
    Initialize `ActionController::Parameters` with `@logging_context`
    Yutaka Kamei authored
    `params` contains `@logging_context` in its instance to notify
    unpermitted parameters including the context through Rails
    Instrumentation API. However, the logging context disappeared when
    `params` is updated with some methods, such as `require`, `slice`,
    `merge`, etc, so the subscriber of `unpermitted_parameters` could not
    get the information.
    
    This patch tries to initialize `Parameters` with `@logging_context`
    where it makes sense to pass the information. The following methods will
    be affected with this patch:
    
    * `require`
    * `deep_dup`
    * `slice`
    * `except`
    * `extract!`
    * `transform_values`
    * `transform_keys`
    * `deep_transform_keys`
    * `select`
    * `reject`
    * `compact`
    * `merge`
    * `reverse_merge`
Loading