Summary
After PR for #802 implements the constantize allowlist fix, this follow-up task verifies the full scope of the params[:model_type] path in AdminSearchQueryWrapper.
Objective
Confirm:
Acceptance Criteria
Verification Plan
- Run the grep commands; confirm no new hits
- Run Brakeman and confirm UnsafeReflection warnings are gone
Client Impact
None (verification + documentation only).
Depends On
#802 (must be merged first)
Summary
After PR for #802 implements the constantize allowlist fix, this follow-up task verifies the full scope of the
params[:model_type]path inAdminSearchQueryWrapper.Objective
Confirm:
constantizeorconst_getpatterns exist in query/search scope that were not covered by Security: Brakeman findings — params-driven constantize (potential RCE) #802Acceptance Criteria
grep -r 'constantize\|const_get\|classify.*params\|camelize.*params' --include='*.rb'across the repoVerification Plan
Client Impact
None (verification + documentation only).
Depends On
#802 (must be merged first)