Skip to content

Conversation

bexsoft
Copy link
Collaborator

@bexsoft bexsoft commented Feb 23, 2023

What does this do?

Disabled Preview button for text files

reivaj05
reivaj05 previously approved these changes Feb 23, 2023
allanrogerr
allanrogerr previously approved these changes Feb 23, 2023
Copy link
Contributor

@allanrogerr allanrogerr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple fix :)

jinapurapu
jinapurapu previously approved these changes Feb 23, 2023
@bexsoft bexsoft dismissed stale reviews from jinapurapu, allanrogerr, and reivaj05 via 9d1a715 February 23, 2023 20:59
@bexsoft bexsoft force-pushed the disable-peview-button branch from 9d1a715 to ee97704 Compare February 23, 2023 21:01
@bexsoft bexsoft merged commit 154974c into minio:master Feb 23, 2023
@bexsoft bexsoft deleted the disable-peview-button branch February 23, 2023 21:25
@jankap
Copy link

jankap commented Mar 24, 2023

Why exactly has the preview been disabled?

@ianroberts
Copy link

What was the reasoning behind this decision? I find the preview option very useful for plain text files as it means I can upload a _README.txt into folders that would otherwise be empty, explaining what sort of files should be dropped there by other people. I suppose I could make a PDF instead, but why disable plain text? Was this to fix some other CVE or something?

@cesnietor
Copy link
Collaborator

I'm trying to find out as well :D. I'm also curious about it. We'll share the context if we end up figuring out. but seems it was for security reasons but the reasons are not explained though. #2021 (comment).

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

Successfully merging this pull request may close these issues.

7 participants