Skip to content

Filtering Job Arguments#277

Merged
rosa merged 18 commits intorails:mainfrom
basecamp:filter-job-args
Jul 30, 2025
Merged

Filtering Job Arguments#277
rosa merged 18 commits intorails:mainfrom
basecamp:filter-job-args

Conversation

@intrip
Copy link
Copy Markdown
Contributor

@intrip intrip commented Jul 29, 2025

Adds a new filter_arguments option that allows you to filter out sensitive data from job arguments.

Note: Currently, only root-level hash keys are supported. If needed, support for key paths, procs, and regular expressions could be added in the future.

@intrip intrip force-pushed the filter-job-args branch from 6b6a4cb to 0a37f71 Compare July 29, 2025 14:32
Adds a new `filter_arguments` option that allows you to filter out sensitive data from job arguments.
This option can be configured globally or per application.

Note: Currently, only root-level hash keys are supported.
If needed, support for key paths, procs, and regular expressions could be added in the future.
@intrip intrip force-pushed the filter-job-args branch from 0a37f71 to cebc01b Compare July 29, 2025 14:42
Comment thread README.md Outdated
So remove the limitation from the Readme
Comment thread README.md Outdated
@rosa
Copy link
Copy Markdown
Member

rosa commented Jul 29, 2025

Thanks @intrip! I don't think it's worth having this per-application. I think this should be a global setting like all the settings listed here.

@intrip intrip force-pushed the filter-job-args branch from 8967fbe to 3f2e086 Compare July 29, 2025 15:51
@intrip
Copy link
Copy Markdown
Contributor Author

intrip commented Jul 29, 2025

Thanks @intrip! I don't think it's worth having this per-application. I think this should be a global setting like all the settings listed here.

Sure! removed it

Comment thread lib/active_job/job_argument_filter.rb Outdated
Comment thread lib/active_job/queue_adapters/resque_ext.rb Outdated
Comment thread lib/mission_control/job_argument_filter.rb Outdated
Comment thread lib/mission_control/job_argument_filter.rb Outdated
Comment thread lib/active_job/queue_adapters/resque_ext.rb Outdated
Copy link
Copy Markdown
Member

@rosa rosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Just left a minor comment but this is ready 🙏

And return an hash or nil
@rosa rosa merged commit 1648c00 into rails:main Jul 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants