sfu-dhil / wphp

Women's Print History Project database front end.
https://womensprinthistoryproject.com/
GNU General Public License v2.0
1 stars 8 forks source link

Fix PDF view permissions #343

Closed andrew-gardener closed 1 year ago

andrew-gardener commented 1 year ago

What isn’t working as expected on the website?

PDFs cannot be viewed by anonymous (not logged in) users

If you encountered an error message, please copy the relevant details or take a screenshot and paste it below.

Unauthorized Access (403)

What is the URL of the page where you encountered the bug?

N/A

What steps did you take before you encountered the bug?

What browser, operating system, and device were you using when you encountered the error?

Firefox

Additional information

N/A

joeytakeda commented 1 year ago

Ha! I just about wrote the same issue earlier this morning :-) and discovered it was an issue with the PDF uploading. When you upload a PDF, you have to set "Public" to true (by default it's false — which may be the thing that ought to be changed)

andrew-gardener commented 1 year ago

Agreed that public should be default.

There is also the issue then that private PDFs shouldn't be listed for users who can't see them anyways

andrew-gardener commented 1 year ago

This will be included in the php upgrade (since its fixed in nines)