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

important: stricter PII handling. Not sharing other user email addresses in emails like invitations #507

Open
TreyWW opened this issue Oct 2, 2024 · 2 comments
Assignees
Labels
idea: accepted This idea has been accepted. Awaiting implementation. idea priority: medium

Comments

@TreyWW
Copy link
Owner

TreyWW commented Oct 2, 2024

How big is this feature?

Small

Describe the problem

At the moment we have checks like {{ request.user.first_name or request.user.email }} has invited you to join xyz! on certain features. This isn't the best habit to be getting into. user first name is acceptable, user email however there may be chances where we leak it without the users action or awareness.

Describe a possible solution

  • warnings to users when actions will be emailing other people, tell the main user that their details will be shared
  • provide more placeholders, OR force users to fill out a first_name instead of having fall backs at all
@TreyWW TreyWW added idea idea: accepted This idea has been accepted. Awaiting implementation. priority: medium labels Oct 2, 2024
@alpha-neutr0n
Copy link

Hi @TreyWW, is this worked upon yet? was planning to give it a shot.

@TreyWW
Copy link
Owner Author

TreyWW commented Oct 5, 2024

Hi @alpha-neutr0n,

Apologies for the late reply. Yep that's fine you can work on this. I'd appreciate it if you could update the issue wherever possible as you make progress. Just helps let me know that you're still working on it and if for whatever reason you get stuck or someone else needs to take over they have some more details.

Thanks! I'll assign you now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea: accepted This idea has been accepted. Awaiting implementation. idea priority: medium
Projects
None yet
Development

No branches or pull requests

2 participants