Skip to content

[#1942] Track missing index files#1943

Merged
Alkarex merged 1 commit intoFreshRSS:devfrom
marienfressinaud:1942-track-missing-index-files
Jun 18, 2018
Merged

[#1942] Track missing index files#1943
Alkarex merged 1 commit intoFreshRSS:devfrom
marienfressinaud:1942-track-missing-index-files

Conversation

@marienfressinaud
Copy link
Copy Markdown
Member

These files must exist for security reasons: they hide contents of their
directories if the webserver isn't well configured and redirect to the
home page.

They were automatically created by ./cli/prepare.php script so it was
annoying to have them in the working tree. Also, the files created by
the script were empty.

Closes #1942

Copy link
Copy Markdown
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

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

This is all of them?

@marienfressinaud
Copy link
Copy Markdown
Member Author

I did git add . so I guess yes ^^

These files must exist for security reasons: they hide contents of their
directories if the webserver isn't well configured and redirect to the
home page.

They were automatically created by `./cli/prepare.php` script so it was
annoying to have them in the working tree. Also, the files created by
the script were empty.
@Alkarex Alkarex merged commit 767ac77 into FreshRSS:dev Jun 18, 2018
@marienfressinaud marienfressinaud deleted the 1942-track-missing-index-files branch June 19, 2018 12:32
@Alkarex Alkarex added this to the 1.11.2 milestone Jan 3, 2021
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this pull request Mar 25, 2021
These files must exist for security reasons: they hide contents of their
directories if the webserver isn't well configured and redirect to the
home page.

They were automatically created by `./cli/prepare.php` script so it was
annoying to have them in the working tree. Also, the files created by
the script were empty.
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.

3 participants