sassoftware / sassoftware/pyviyatools

Enhancement suggestion: add object path to output of explainaccess.py with -u argument

Open
#6 0 comments 0 reactions 1 assignee View on GitHub

@dwstern is already working on this.

Since Jan 15, 2019.

Dominant language
Python
Stars
50
Forks
42
PR merge metrics
No merged PRs in 30d

Description

explainaccess.py can report the effective access of a folder (identified by its folder path using the -f parameter) or other object (which can be a folder, a report etc, identified by its objectURI using the -u parameter).

The optional -p parameter tells explainaccess.py to prefix each row of output with the path (the same path that was passed in, if the -f parameter was used, or the objectURI if the -u parameter was used).

Suggest enhancing the code so that if the object whose permissions we want to explain is identified using -u objectURI, use the new getpath() sharedfunction to report the object's actual path, perhaps suffixed with its name and type, in addition to or instead of the object URI. Need to make sure the output columns are consistent between -f and -u use cases.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.