Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a UUID when logging #72

Open
egreer opened this issue Oct 10, 2024 · 0 comments
Open

Add a UUID when logging #72

egreer opened this issue Oct 10, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@egreer
Copy link
Contributor

egreer commented Oct 10, 2024

Log files that get very long or have multiple allow grants it can be tough to identify the access grant enabled with the corresponding disable.

2024-10-10T03:41:17.094931  INFO: [PHI Access Log] [Foo] PHI Access Enabled for 'Bar': reason
2024-10-10T03:41:17.194931  INFO: [PHI Access Log] [Foo] PHI Access Enabled for 'Bar2': reason
.....
2024-10-10T03:41:18.094931  INFO: [PHI Access Log] [Foo] PHI access disabled for 'Bar2': reason
2024-10-10T03:41:18.101662  INFO: [PHI Access Log] [Foo] PHI access disabled for 'Bar'

Proposal: Generate a UUID with each grant that we can use for the log.

2024-10-10T03:41:17.094931  INFO: [PHI Access Log] [Foo] [13cf87b3-c17d-4c09-a84b-6868bf149973] PHI Access Enabled for 'Bar': reason
2024-10-10T03:41:17.194931  INFO: [PHI Access Log] [Foo] [0ba49a67-4e3e-4b4d-85b5-0263bae7a73c] PHI Access Enabled for 'Bar2': reason
.....
2024-10-10T03:41:18.094931  INFO: [PHI Access Log] [Foo] [0ba49a67-4e3e-4b4d-85b5-0263bae7a73c] PHI access disabled for 'Bar2'
2024-10-10T03:41:18.101662  INFO: [PHI Access Log] [Foo] [13cf87b3-c17d-4c09-a84b-6868bf149973] PHI access disabled for 'Bar'

This would update the phi context pushed onto the stack:

{
   phi_access_allowed: true,
   user_id: user_id,
   reason: reason,
   id: SecureRandom.uuid
 }
@egreer egreer added the enhancement New feature or request label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant