Skip to content
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

Fix get_intensity permissions #250

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

will-moore
Copy link
Member

See https://www.openmicroscopy.org/community/viewtopic.php?f=4&t=8698

To test:

  • Choose another user's image in your non-default group and try to query pixel intensity.

@jburel jburel added this to the 0.7.0 milestone Mar 14, 2019
@pwalczysko
Copy link
Member

Tested on eel (could repeat two situations on outreach, user views data in their non-default group of other users, and admin views data in their non-default group of other users).
All works as expected now

Ready to merge FMPOV

@jburel
Copy link
Member

jburel commented Mar 15, 2019

@pwalczysko did you check with public user too?

@pwalczysko
Copy link
Member

@jburel no, but tried this additional test just now: add public user to read-only-1, this is public user's non-default group. Then view image of another user as public user in read-only-1. All works fine.

@jburel jburel merged commit 74b6f24 into ome:master Mar 19, 2019
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.

3 participants