roidrage / lograge

An attempt to tame Rails' default policy to log everything.
http://www.paperplanes.de/2012/3/14/on-notifications-logsubscribers-and-bringing-sanity-to-rails-logging.html
MIT License
3.47k stars 300 forks source link

Unable to Filter ActionCable Params #356

Open sirwolfgang opened 2 years ago

sirwolfgang commented 2 years ago

I am unable to filter action cable params.

method= path= format= params={"token"=>"...."} controller=ApplicationCable::Connection action=connect status=200 duration=216.54

They come in on the data attribute here but don't seem overloadable in custom_options by setting both params or data

iloveitaly commented 2 years ago

Want to submit a PR with a patch? This PR may be related https://github.com/roidrage/lograge/pull/310

On Tue, Aug 2, 2022 at 1:31 PM Zane Wolfgang Pickett < @.***> wrote:

I am unable to filter action cable params.

method= path= format= params={"token"=>"...."} controller=ApplicationCable::Connection action=connect status=200 duration=216.54

They come in on the data attribute here https://github.com/roidrage/lograge/blob/master/lib/lograge/rails_ext/action_cable/connection/base.rb#L15 but don't seem overloadable in custom_options by setting both params or data

— Reply to this email directly, view it on GitHub https://github.com/roidrage/lograge/issues/356, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABE2R3U3MPQ6D6H5JCJRPDVXFZSJANCNFSM55MJT76Q . You are receiving this because you are subscribed to this thread.Message ID: @.***>

sirwolfgang commented 2 years ago

I was unable to patch it locally