Skip to content

53 enhanced user privacy#55

Open
ThrawnCA wants to merge 7 commits into
data-govt-nz:masterfrom
ThrawnCA:53-enhanced-user-privacy
Open

53 enhanced user privacy#55
ThrawnCA wants to merge 7 commits into
data-govt-nz:masterfrom
ThrawnCA:53-enhanced-user-privacy

Conversation

@ThrawnCA
Copy link
Copy Markdown
Contributor

@ThrawnCA ThrawnCA commented Sep 8, 2022

Improve the privacy of user data. If ckan.public_user_details is False, then user profiles should only be visible to their owners or relevant admins.

@markstuart
Copy link
Copy Markdown
Contributor

This looks great @ThrawnCA, love your work!

Could you add something to the README describing this behaviour? Happy to merge after that :)

@ThrawnCA
Copy link
Copy Markdown
Contributor Author

README is updated; feedback welcome.

@markstuart
Copy link
Copy Markdown
Contributor

After reviewing this again, I'm a little concerned that this feature could be too opinionated for many consumers. Would it make sense to provide a flag like ckanext.security.admin_only_user_details? For example, in our application all logged in users should be able to see other users of the system, but public users are not allowed to, so the standard ckan.public_user_details flag is fine.

@ThrawnCA
Copy link
Copy Markdown
Contributor Author

ThrawnCA commented Feb 9, 2023

For example, in our application all logged in users should be able to see other users of the system, but public users are not allowed to, so the standard ckan.public_user_details flag is fine.

I assume that you don't allow public registration, then?

Why would one regular user need to see another's profile?

@ThrawnCA
Copy link
Copy Markdown
Contributor Author

@markstuart How about a middle ground?

Suppose there were a flag to allow any member of an organisation - not just an org admin - to view the user profiles of other members of the same organisation. That way, if you really want to allow every account on the site to view each other's profiles, you can create an umbrella organisation and make them all members of it.

That would also give more flexibility if you only want most profiles to be visible to each other, retaining a few exceptions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants