-
Notifications
You must be signed in to change notification settings - Fork 3
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
App summary of timed count doesn't match data in the database #719
Comments
issue related to sensitive species #691 |
@johnvanbreda, the app uses ES to fetch user samples. The ES doesn't return me some of those 8 subsamples belonging to the E.g. running the below {
"size": 10000,
"query": {
"bool": {
"must": [
{
"match": {
"id": "26995680"
}
}
]
}
}
} but this one doesn't: {
"size": 10000,
"query": {
"bool": {
"must": [
{
"match": {
"id": "26995666"
}
}
]
}
}
} |
@kazlauskis using es-samples, it will apply default filtering and sensitivity blurring etc. Can you try the same query, but add this to the JWT payload: That tells the system that the user is looking at just their own data, so allows full precision data. |
@johnvanbreda the JWT is created by the identity manager, i.e. Drupal, where the scopes=roles. I could create a role, or modify some Drupal module to include this extra scope to app users, but I wonder if there is an easier way here, like a request header or a param that I could change in the client (app or website) on the fly. A client with a single user authentication connection may want to run global and sometimes user-specific queries, so it is good to have some way to do that without changing the JWT. |
Raised by a user. They have provided a screenshot of the details of the details held in the app for a 15 minute timed count. All the details match the raw data held on the warehouse except for 1 taxon, which shows a single record on the app summary, but 8 records in the warehouse. The records have different external keys and different sample entered_sref values so are not duplicates.
@kazlauskis I'll email the details over for when you get round to this as I'm not sure if the taxa are supposed to be sensitive.
The text was updated successfully, but these errors were encountered: