Add PII correlation tool#2049
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cellio
left a comment
There was a problem hiding this comment.
Tested on my dev server. The tool looks good, a user with a different email domain shows as different (good), all the IPs look the same. I tried using a VPN to log in one of my users but it still showed up all red; I'm not sure if localhost overrides VPN or the weekly job needs to run or something else. We should test this on the dev server before deploying to make sure we aren't unintentionally implicating a lot of people who aren't really on the same IPs.
For email addresses, I only saw "unknown number of users", even when I made up fake domains. What's an example of a domain that should produce some other description there?
Most of the code here is above my head; someone else should definitely review.
These stats are recalculated every week and cached in between. To generate the initial cache, run
Localhost probably overrides VPN. You can tell if it's a localhost address - for IPv4 that's 127.0.0.1; although it's hashed you'll be able to see that the middle two octets are the same and the first and last are different. For IPv6 it's |
Yep, apparently I completely missed that |
](https://private-user-images.githubusercontent.com/5557942/594796375-d8b7ba6a-2a8d-49b1-b13e-44447282bb82.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Nzk2MjIzODYsIm5iZiI6MTc3OTYyMjA4NiwicGF0aCI6Ii81NTU3OTQyLzU5NDc5NjM3NS1kOGI3YmE2YS0yYThkLTQ5YjEtYjEzZS00NDQ0NzI4MmJiODIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI2MDUyNCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNjA1MjRUMTEyODA2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9OGVmOGJhZWIwYjQzYmI2MmM2MWYwZmY5M2Q1YTdmZmQwMjE3MWY0NTVmNjgxYmYwNDhmMjU2OWE3MjA4ZjYyMyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmcmVzcG9uc2UtY29udGVudC10eXBlPWltYWdlJTJGcG5nIn0.v9dg2DvzOhEl9EULNiMk-0ZGATcTHu_u2lj3I9g0EJI)
Closes #2000.
Adds a tool for moderators to correlate user PII, as described in the linked issue.