Limit scope of lookups in article (to prevent potential corruption) #270

Open
opened 2019-11-17 16:23:35 +00:00 by matthijskooijman · 0 comments
matthijskooijman commented 2019-11-17 16:23:35 +00:00 (Migrated from github.com)

Currently, the article datatype does some e.g. comment lookups based on ids, without limiting to the relevant section. This works fine when the id passed is in the right section, but the id passed might belong to something else, breaking things.

See https://github.com/PlanBCode/hypha/pull/258#discussion_r301532916 for where in the code (which is currently being refactored) this is.

Currently, the article datatype does some e.g. comment lookups based on ids, without limiting to the relevant section. This works fine when the id passed is in the right section, but the id passed might belong to something else, breaking things. See https://github.com/PlanBCode/hypha/pull/258#discussion_r301532916 for where in the code (which is currently being refactored) this is.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
harmen/hypha#270
No description provided.