-
Notifications
You must be signed in to change notification settings - Fork 0
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
Check for equivalence of term labels for duplicate TermURLs across nodes #68
Comments
I would vote to bump this from the backlog and give it a |
Sure, that sounds okay to me. |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
|
federation-api/app/api/crud.py
Lines 163 to 165 in 9d685db
If we overwrite previous instances here, would it be reasonable to check first if the term is the same as the one we've already stored? If we ever encounter a situation where some jokester has a neurobagel node where they just make up their own TermURLs for our terms, we could end up with a tricky to debug situation if all that determines what we return here is the order in which we unpack the responses from nodes.
Originally posted by @surchs in #65 (comment)
The text was updated successfully, but these errors were encountered: