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

Support access control via adding 'controlled' to folder name instead of file name #494

Closed
junjun-zhang opened this issue Feb 20, 2018 · 0 comments
Assignees

Comments

@junjun-zhang
Copy link

This is a feature request in preparation of hosting large amount of controlled access data files from PCAWG study.

Without this feature, all controlled access files have to be renamed to add controlled in the file name, which is not desirable as files are likely referenced by the original name already.

In the following example, files: file_2.controlled.txt, file_3, file_4 and file_5 are controlled access.

PCAWG
 |__ data_folder_1
 |    |__ file_1
 |    |__ file_2.controlled.txt
 |__ data_folder_2.controlled
      |__ file_3
      |__ file_4
      |__ data_folder_3
           |__ file_5

Also, currently, controlled access file is only visible (listed) when user logs in. It would be greatly helpful, if all files are alway visible regardless controlled or not. Only download controlled file requires logging in.

@Fgerthoffert Fgerthoffert added the next AB:2:Ready For Implementation label Feb 27, 2018
@Fgerthoffert Fgerthoffert added this to the DCC - Sprint 184 milestone Feb 27, 2018
@Fgerthoffert Fgerthoffert removed this from the DCC - Sprint 184 milestone Mar 15, 2018
@Fgerthoffert Fgerthoffert added backlog AB:1:Backlog and removed next AB:2:Ready For Implementation labels Apr 5, 2018
@andricDu andricDu assigned lepsalex and unassigned KevinFHartmann May 7, 2018
@lepsalex lepsalex added next AB:2:Ready For Implementation in progress AB:3:In Progress review AB:4:Review and removed backlog AB:1:Backlog next AB:2:Ready For Implementation in progress AB:3:In Progress labels May 9, 2018
@lepsalex lepsalex removed the review AB:4:Review label May 14, 2018
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

No branches or pull requests

4 participants