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

DRP Shell - Stop cat abuse #176

Closed
adhdtech opened this issue Jun 1, 2021 · 1 comment
Closed

DRP Shell - Stop cat abuse #176

adhdtech opened this issue Jun 1, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@adhdtech
Copy link
Owner

adhdtech commented Jun 1, 2021

The cat function in the DRP shell is used to retrieve data, but it also triggers the execution of functions which mimic directories. The problem is that cat also triggers functions which cause change. Need to come up with a way to differentiate between functions which mimic directories and those which do not. If a function capable of change receives a "read-only" command (cat, ls, etc) then the response will differ from that of a write command. Common sense stuff that should have been addressed long ago.

Need to look at how PowerShell does it and implement a similar scheme. This is the reason the method cliGetPath was implemented in the beginning. Or maybe use REST verbs as attributes in the pathCmd method and get rid of the cliGetPath method in favor of this.

@adhdtech adhdtech added the enhancement New feature or request label Jun 1, 2021
@adhdtech adhdtech added this to the DRP Shell VDM Applet milestone Jun 1, 2021
@adhdtech adhdtech self-assigned this Jun 1, 2021
@adhdtech
Copy link
Owner Author

Implemented "exec" method in DRP Shell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant