freshshell / fresh

Keep your dotfiles fresh.
https://freshshell.com/
1.17k stars 87 forks source link

Example installation vulnerable to MITM leading to the execution of arbitrary code #139

Closed ypid closed 7 years ago

ypid commented 7 years ago

Hey guys

I just happend to find this project while look for something else and stumbled across this line:

bash -c "`curl -sL get.freshshell.com`"

which can not be left without opening an issue. I was astonished to not find an issue already open for this.

There are multiple problems with this:

That just shows me that you guys seem to favor "coolness" over the absolute minimum of security. For example, you could have decided to use the github URL directly in your example but you did go with get.freshshell.com.

I encourage you to work on getting a badge here https://bestpractices.coreinfrastructure.org/ and fix any criteria you currently don’t meet. Dotfiles are a core part of a persons work environment.

This repository seems to have 700 stars. What is going one? How many of you guys install software like this?

PS: I just saw that the last commit was back in 2014. Anyway, the readme still stats that it is maintained so.