-
Notifications
You must be signed in to change notification settings - Fork 480
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
Sanitize comment block for under 13 shares #22204
Conversation
5bcd210
to
c2ced6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet. Planning to do the admin case before merging this PR, or in a follow-up?
Before merging; I figure the change itself will be small enough relative to the rest of this that it was worth getting a review beforehand |
Improved and fixed tests; improved by adding tests for admin case, fixed by using a better stubbing method. PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
When viewing a project created by an under-13 user, only the user themselves should be able to view comment blocks; all other users should see a sanitized version of the source.
TODO: need to figure out which version admins should see, and add a test for that case