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

Permissions are wrong for projects #25

Open
morungos opened this issue Dec 13, 2013 · 4 comments
Open

Permissions are wrong for projects #25

morungos opened this issue Dec 13, 2013 · 4 comments
Assignees
Labels

Comments

@morungos
Copy link
Member

Ivan reported this: "I found some issues from the the webapp I can not see the project table, also if I click on a sample I get 'Sorry, you're not authorized to view this page'."

@ghost ghost assigned morungos Dec 13, 2013
@morungos
Copy link
Member Author

Asked Ivan: "How did you log in? As iborozan or as admin?" Realized, I might even be able to work this out from the logs. If not, I probably should be able to.

@morungos
Copy link
Member Author

And yes, I can replicate this. The samples are listed, but no projects are listed. Displaying any individual sample gets this effect though. Also, there's no UI to allow constraining to specific projects, so the permissions system never really allowed that flexibility in the first place. We should probably start by relaxing all permissions to get viability.

@morungos
Copy link
Member Author

Oh well, this part of CaPSID is a total mess. Old CaPSID used DBRefs, which have been deprecated for years, and which don't work that well. I can't even see how these were ever created. Also, the code appears to use LDAP to query roles, even though the database also stores them. And they are actually used extensively in the code base, so it isn't all that clear how well we can just hack that code out.

@morungos
Copy link
Member Author

Well, with investigation, there seems to be a workaround. Projects can be assigned users, but it's in a weird and hidden part of the interface. Choose the Edit button for a selected project and you can add users. Of course, part of the problem is that samples shouldn't be displayed unless you have project permissions, which is is actually the problem here: stuff that isn't supposed to be accessible actually is.

morungos added a commit that referenced this issue Jun 6, 2014
…s the primary scope for now. Should probably handle permissions elsewhere too. #25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant