apsislabs / apsislabs/phi_attrs

Don't require phi_access logging for `new` models

Open
#39 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Ruby
Stars
31
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Sometimes you just want a transitory `MyModel.new(params).method_call`

It's super annoying to have to assign a variable just to allow phi on the params you're already touching and are unprotected anyway.

We shouldn't 'engage' PHI_Attrs until it's been saved to the database.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the PHI_Attrs behavior triggered when a model is initialized and when it is saved. Verify that MyModel.new(params).method_call does not require PHI access logging before persistence, while access to a saved model remains logged.

Written by the indexing model from the issue text.

Assessment

Tech stack
rails, ruby
Domain
backend, databases, security
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.