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

Restricted File Download: "Download (selected files)" button above files table is not properly handling restricted files #2722

Closed
sbarbosadataverse opened this issue Nov 4, 2015 · 17 comments
Assignees
Labels
Feature: File Upload & Handling Feature: Permissions Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner
Milestone

Comments

@sbarbosadataverse
Copy link

If I navigate to this dataset, when not logged in: https://dataverse.harvard.edu/dataset.xhtml?persistentId=hdl%3A1902.1%2F00169

and I select the restricted files and select download at the top right of the file list--because it shows up as an option and doesn't disappear when I select restricted files, it throws the attached error instead of a message saying "files can't be downloaded see terms of use"

screen shot 2015-11-04 at 12 29 50 pm

Additionally, the Terms of Use are popping up which should not be happening.

@pdurbin
Copy link
Member

pdurbin commented Nov 4, 2015

This may be a duplicate of #2719.

@landreev
Copy link
Contributor

landreev commented Nov 4, 2015

No, I don't think this has anything to do/in common with #2719.

@mercecrosas mercecrosas modified the milestone: In Review Nov 30, 2015
@eaquigley
Copy link
Contributor

@sbarbosadataverse do you mind trying this again and seeing if it is still happening? i believe this was before we made changes to the dataset page. thanks!

@sbarbosadataverse
Copy link
Author

sure will!

On Tue, Jan 5, 2016 at 1:46 PM, Elizabeth Quigley [email protected]
wrote:

@sbarbosadataverse
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_sbarbosadataverse&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=8R6PzVqt1PEocauQgZMGXsGz29-nb19M7eqlo1d8EVs&m=MiIxyiKP2HxgQiNhstxu7jV31uzeb2TEXpb5_UVb124&s=JIhoDF2r9RuoUjmRuJlUuyw_opMht1lMjxLqbDY5HSM&e=
do you mind trying this again and seeing if it is still happening? i
believe this was before we made changes to the dataset page. thanks!


Reply to this email directly or view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IQSS_dataverse_issues_2722-23issuecomment-2D169094044&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=8R6PzVqt1PEocauQgZMGXsGz29-nb19M7eqlo1d8EVs&m=MiIxyiKP2HxgQiNhstxu7jV31uzeb2TEXpb5_UVb124&s=mrulFo31z_w7Uzjiv-vvmGXxsy65Hfa5QEcahjTLkjE&e=
.

Sonia Barbosa
Manager of Data Curation, IQSS Dataverse Network
Manager of the Murray Research Archive, IQSS
Data Science
Harvard University

Dataverse 4.0 is now available for use!
http://dataverse.harvard.edu

All test dataverses should be created in 4.0 Demo!
http://dataverse-demo.iq.harvard.edu/

Join our Dataverse Community!
https://groups.google.com/forum/#!forum/dataverse-community

@pdurbin
Copy link
Member

pdurbin commented Jan 5, 2016

This is still happening. For example: https://dataverse.harvard.edu/api/access/datafiles/104548

@sbarbosadataverse
Copy link
Author

@eaquigley just tested and still happens.
The "download" option is still available on the top of the files list, when not logged on, and select a "Restricted" file that doesn't allow "request access to data"

Glass fish error shows up after you see the "murray terms" of agreement and of course, you shouldn't even get that far

@eaquigley
Copy link
Contributor

@sbarbosadataverse ah! not logged in is the account state for this. adding that to the original issue text. thanks!

@scolapasta scolapasta removed their assignment Jan 27, 2016
@mheppler mheppler added Feature: File Upload & Handling Feature: Permissions UX & UI: Design This issue needs input on the design of the UI and from the product owner labels Jan 28, 2016
@scolapasta scolapasta modified the milestone: Not Assigned to a Release Jan 28, 2016
@mheppler mheppler changed the title when attempting to download restricted files: support message shows glassfish error instead of something useful Restricted File Download: Error message shows 403/glassfish error instead of something useful Jan 29, 2016
@sbarbosadataverse
Copy link
Author

@scolapasta @mheppler
This needs to be modified that the download is showing up and available to click for "restricted
" files<---this should not be possible!
resulting in the error below, and actively modifying the guestbook.

The other bug described by Tim (Brain Genomics Dataverse)

  1. Open a private browsing tab
  2. Navigate to
    https://dataverse.harvard.edu/dataset.xhtml?persistentId=doi:10.7910/DVN/25833
  3. Click the checkbox next to a restricted file.
  4. Click the Download button (not the Request Access button)

Expected Result:
User is not authenticated and will not be presented with the Guestbook. The
user should be instructed to click on the Request Access button, or the
Download button should not be present.

Actual Result:
User is presented with the Guestbook. The user can fill out the form and
submit. The server responds with a 403 Forbidden, but the Guestbook entry
is still saved.

By the way--this is likely not contained to Tim's dataverse--but likely every dataverse that has restricted files and a guestbook.

@mheppler
Copy link
Contributor

mheppler commented Feb 2, 2016

OK, to summarize, as I understand it.

The "Download" button at the top of the files table on the dataset page, which is intended for downloading "selected files", is not properly handling the use case for restricted files being selected.

@sbarbosadataverse sbarbosadataverse changed the title Restricted File Download: Error message shows 403/glassfish error instead of something useful Restricted File Download: The "Download" button at the top of the files table on the dataset page, which is intended for downloading "selected files", is not properly handling the use case for restricted files being selectedError message shows 403/glassfish error instead of something useful Feb 3, 2016
@mheppler mheppler changed the title Restricted File Download: The "Download" button at the top of the files table on the dataset page, which is intended for downloading "selected files", is not properly handling the use case for restricted files being selectedError message shows 403/glassfish error instead of something useful Restricted File Download: "Download (selected files)" button above files table is not properly handling restricted files Feb 3, 2016
@scolapasta scolapasta added this to the 4.2.4 milestone Feb 4, 2016
@sekmiller
Copy link
Contributor

Based on discussions with Liz and Gustavo I have implemented the following:

If the user selects only files that are restricted he will be shown a popup saying that none of the selected can be downloaded. If the dataset allows request access that will be noted in the popup.

If the user selects both files that he can and cannot download, he will be shown a popup which lists by name those files that cannot be downloaded with an option to continue to download those files for which he does have permission or to cancel.

If he chooses to continue the guestbook/terms will be shown next or if not required the download of permitted files will continue directly.

@sekmiller
Copy link
Contributor

@eaquigley can you review the bundle and make changes or pass to Kevin.

@sekmiller sekmiller assigned eaquigley and unassigned sekmiller Feb 8, 2016
@eaquigley
Copy link
Contributor

@sekmiller can do. could you comment on the pull request for this feature with this issue # so its easier to find. thanks!

eaquigley added a commit that referenced this issue Feb 8, 2016
updated text for this workflow to be more user friendly. Issue #2722
@eaquigley
Copy link
Contributor

text updated in the bundle. passing to @kcondon. could i get another build on dataverse-internal to see how the text looks in the UI? thanks!

@eaquigley eaquigley assigned kcondon and unassigned eaquigley Feb 8, 2016
@kcondon
Copy link
Contributor

kcondon commented Feb 8, 2016

OK, popups are correct. Download does not work in the case where you want to download the unrestricted files when both restricted and unrestricted are selected, they don't actually download.

@kcondon kcondon assigned sekmiller and unassigned kcondon Feb 8, 2016
@eaquigley
Copy link
Contributor

Had @sbarbosadataverse try out this new workflow. Everything looks good to her except for how difficult it is to identify the names of the restricted files that won't be downloaded. @sbarbosadataverse also stated that no matter the # of restricted files selected, all names need to appear either in a column style or the way it currently is with more formatting (like: File 1, File 2, File 3) and a way for them to stand out more.

@sekmiller @scolapasta @kcondon @mheppler

@sekmiller
Copy link
Contributor

I have formatted the requested restricted files as a list for readability.

The code has been pushed to the branch.

@sekmiller sekmiller assigned kcondon and unassigned sekmiller Feb 9, 2016
@kcondon
Copy link
Contributor

kcondon commented Feb 9, 2016

Works as described, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: File Upload & Handling Feature: Permissions Type: Bug a defect UX & UI: Design This issue needs input on the design of the UI and from the product owner
Projects
None yet
Development

No branches or pull requests

9 participants