Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix-docker-pecl: Fix Dockerfile after pear.php.net hack #6641

Merged
merged 1 commit into from
Jan 25, 2019
Merged

fix-docker-pecl: Fix Dockerfile after pear.php.net hack #6641

merged 1 commit into from
Jan 25, 2019

Conversation

colin-campbell
Copy link
Contributor

pear.php.net was distributing a hacked go-pear.phar and is now offline. Change Dockerfile to get known good phar from github.
see:
https://blog.rapid7.com/2019/01/22/php-extension-and-application-repository-pear-compromise-what-you-need-to-know/

@snipe
Copy link
Owner

snipe commented Jan 25, 2019

To be honest, i’m Not sure what we even need pear for....

@colin-campbell
Copy link
Contributor Author

For building the deprecated mcrypt, for the LegacyEncrypter?

@snipe
Copy link
Owner

snipe commented Jan 25, 2019

Ah, yeah - maybe some of the stuff in the MycryptEncrypter. Ugh. And there's no way out of that, if we want to still allow people on very old installs to be able to upgrade.

Maybe it makes sense to move the recrypter into a separate package, so the 95% of the people who will never use it don't have to carry around that old dependency.

@snipe snipe merged commit f8d9301 into snipe:develop Jan 25, 2019
@colin-campbell
Copy link
Contributor Author

I will try to replace the mcrypt RIJNDAEL_* with the openssl AES_* equivalents, if I get some time. Mcrypt isn't coming back.

@snipe
Copy link
Owner

snipe commented Jan 26, 2019

Oh of course it’s not - the only reason we keep the legacy stuff there is because if someone has a very old install (which used mcrypt) and had anything encrypted in it (encrypted custom fields, LDAP password, etc), without running the recrypter, that info in the database would be irretrievably lost when they upgrade. It’s a small percentage of people as far as I can tell, but upgrading without recrypting would be data destructive. Believe me, I wouldn’t have built that abomination if I didn’t have to.

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.

None yet

2 participants