Project

General

Profile

Bug #8335

Use of salt in bcrypt is deprecated in PHP7

Added by Yannick Warnier about 5 years ago. Updated about 5 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Installation / Migration
Target version:
Start date:
14/07/2016
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

When installing 1.11.0-alpha on PHP7 with bcrypt encryption, I get this:

Deprecated: password_hash(): Use of the 'salt' option to password_hash is deprecated in /var/www/chamilo/www/vendor/symfony/security/Core/Encoder/BCryptPasswordEncoder.php on line 81

Deprecated: password_hash(): Use of the 'salt' option to password_hash is deprecated in /var/www/chamilo/www/vendor/symfony/security/Core/Encoder/BCryptPasswordEncoder.php on line 81

Warning: Cannot modify header information - headers already sent by (output started at /var/www/chamilo/www/main/install/index.php:392) in /var/www/chamilo/www/main/inc/lib/template.lib.php on line 1022

Warning: Cannot modify header information - headers already sent by (output started at /var/www/chamilo/www/main/install/index.php:392) in /var/www/chamilo/www/main/inc/lib/template.lib.php on line 1024

Warning: Cannot modify header information - headers already sent by (output started at /var/www/chamilo/www/main/install/index.php:392) in /var/www/chamilo/www/main/inc/lib/api.lib.php on line 7524

Selecting SHA1, this error does not occur.

History

#1

Updated by Julio Montoya about 5 years ago

Do we support officially php 7.0?

#2

Updated by Julio Montoya about 5 years ago

well this is an issue with "sonata user bundle" and "friendsofsymfony/user-bundle"

#3

Updated by Julio Montoya about 5 years ago

it was fixed in symfony 3 according to http://stackoverflow.com/questions/34549311/fosuserbundle-bcryptpasswordencoder-salting

but those bundles require a different version of symfony2

#4

Updated by Yannick Warnier about 5 years ago

Julio Montoya wrote:

Do we support officially php 7.0?

I think we can do that... I am trying to resolve a few issues with behat, but I think we can "officially beta support PHP 7" :-)

#5

Updated by Julio Montoya about 5 years ago

Yannick Warnier wrote:

Julio Montoya wrote:

Do we support officially php 7.0?

I think we can do that... I am trying to resolve a few issues with behat, but I think we can "officially beta support PHP 7" :-)

then its ok that warning appear :D
This was fixed in other versions of Symfony2

#6

Updated by Julio Montoya about 5 years ago

  • Assignee changed from Julio Montoya to Yannick Warnier

we can silent that warning too ...

#7

Updated by Yannick Warnier about 5 years ago

  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 0 to 50

OK, please silence this option (I don't know where to do it). If it's solved in future versions of Symfony and we're going to move to a superior version in the future anyway, I think it's acceptable.

#8

Updated by Julio Montoya about 5 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from Julio Montoya to Yannick Warnier
#9

Updated by Yannick Warnier about 5 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 50 to 100

Could for me. I added a comment to the commit ;-)

Also available in: Atom PDF