Skip to content
This repository has been archived by the owner on Apr 27, 2022. It is now read-only.

Add remaining operations to picasso #46

Open
chemistry-sourabh opened this issue Jan 13, 2017 · 7 comments
Open

Add remaining operations to picasso #46

chemistry-sourabh opened this issue Jan 13, 2017 · 7 comments
Milestone

Comments

@chemistry-sourabh
Copy link
Contributor

There are certain admin level operations that were not added to picasso due to security reasons. These operations should be added as path of the rest API, but must work only for admins.

@ravisantoshgudimetla
Copy link
Contributor

Can you list them here and what you should think of the flow.

@chemistry-sourabh
Copy link
Contributor Author

Operations like db ls which use the einstein code directly without sending a REST request to picasso. This will allow us to detach the CLI as you previously pointed out. Also these commands should work only for admins and end users will get a 403

@ravisantoshgudimetla
Copy link
Contributor

Operations like db ls which use the einstein code directly without sending a REST request to picasso.

Can you be a bit more elaborate. I am kind of lost here. So, are you telling that once we separate will it work or it won't?

@chemistry-sourabh
Copy link
Contributor Author

Right now db ls is directly calling the method in BMI object. After adding to picasso then it will become an RPC call.

@apoorvemohan
Copy link
Collaborator

@chemistry-sourabh Can you document these TODO operations somewhere?

@chemistry-sourabh
Copy link
Contributor Author

All the operations that are calling BMI directly in CLI

@naved001 naved001 added this to the Redesign milestone Nov 22, 2017
@apoorvemohan
Copy link
Collaborator

@chemistry-sourabh can we close this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants