Skip to content

Conversation

@oldskool
Copy link
Contributor

In the Blog/Auth tutorial, the deprecated AuthComponent::password method was still referenced, while this is now a deprecated function in favor of the Security::hash method. Adjusted the docs to reflect this change.

@ravage84
Copy link
Member

I see what you did there...
You replaced the old call with the one that is used in it now:
http://api.cakephp.org/2.4/source-class-AuthComponent.html#802

http://book.cakephp.org/2.0/en/core-utility-libraries/security.html#Security::hash
None of the examples there show it like you used in your change.
That could make people wondering why it is used like that in this example while nowhere in the other examples.

@ravage84
Copy link
Member

I guess you should use the SimplePasswordHasher example from here:
http://book.cakephp.org/2.0/en/core-libraries/components/authentication.html#hashing-passwords

@markstory
Copy link
Member

This update is a step in rhe right direction. Using the password hasher classes would be even better.

ravage84 added a commit to ravage84/cakephpdocs that referenced this pull request Nov 17, 2013
@ADmad
Copy link
Member

ADmad commented Nov 17, 2013

Closing in favor of #896.

@ADmad ADmad closed this Nov 17, 2013
@oldskool
Copy link
Contributor Author

Fair enough. I missed the SimplePasswordHasher, but that looks more suitable for this situation. 👍

@okonomi okonomi reopened this Dec 2, 2013
@okonomi okonomi closed this Dec 2, 2013
@okonomi
Copy link
Member

okonomi commented Dec 2, 2013

Sorry, I was mistaken operation.

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.

5 participants