Skip to content

Commit

Permalink
Major update on security, improved UI and bug fix.
Browse files Browse the repository at this point in the history
  • Loading branch information
prasathmani committed Nov 19, 2022
1 parent 4d8d4a3 commit 8de16a4
Show file tree
Hide file tree
Showing 4 changed files with 366 additions and 461 deletions.
6 changes: 2 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,13 @@ Default username/password: **admin/admin@123** and **user/12345**.

To enable/disable authentication set `$use_auth` to true or false.

:information_source: Rename the `config-sample.php` file into `config.php` to use configuration, it is an additional configuration file, Feel free to remove completely this file and configure "tinyfilemanager.php" as a single file application.
:information_source: Add your own configuration file `config.php` in the same folder to use as additional configuration file.

### :loudspeaker: Features

- :cd: Open Source, light and extremely simple
- :iphone: Mobile friendly view for touch devices
- :information_source: Basic features likes Create, Delete, Modify, View, Quick Preview, Download, Copy and Move files
- :information_source: Basic features likes Create, Delete, Modify, View, Download, Copy and Move files
- :arrow_double_up: Ajax Upload, Ability to drag & drop, upload from URL, multiple files upload with file extensions filter
- :file_folder: Ability to create folders and files
- :gift: Ability to compress, extract files (`zip`, `tar`)
Expand Down Expand Up @@ -79,8 +79,6 @@ DockerHub: [https://hub.docker.com/r/tinyfilemanager/tinyfilemanager](https://hu

#### How to change config within docker

**Important!!!** First, you can copy `config-sample.php` to `config.php`, and must modify this following config

Origin:

```php
Expand Down
123 changes: 0 additions & 123 deletions config-sample.php

This file was deleted.

4 comments on commit 8de16a4

@michacassola
Copy link

Choose a reason for hiding this comment

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

I don't know how removing the config sample that I relied on makes things more simple, as it makes things harder for me right now having to rewrite my script, but there is nothing I can do about it is there?

@prasathmani
Copy link
Owner Author

Choose a reason for hiding this comment

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

@michacassola , I have removed sample-config.php to make file manager simple, as it is a single file manager. still if you have config.php, it will work.
https://github.com/prasathmani/tinyfilemanager/blob/master/tinyfilemanager.php#L135

@michacassola
Copy link

@michacassola michacassola commented on 8de16a4 Dec 14, 2022

Choose a reason for hiding this comment

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

Please don't misunderstand me, I am thankful for this great piece of FOSS!

But you could have removed screenshot.gif instead which is very annoying and quite large with more than 2mb, it doesn't have to be on the same repo to be inserted into the README.MD.
Also the Dockerfile doesn't need to be there, could be in another repo as well.
And SECURITY.MD is another thing that doesn't have to be there...

So 3 files that would make more sense regarding decluttering right there.

@prasathmani
Copy link
Owner Author

Choose a reason for hiding this comment

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

Dockerfile and SECURITY.MD are required files, but screenshot.gif can be compressed into small size or move to another repo.

Please sign in to comment.