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.

Sign up for GitHub

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

Jump to bottom

feat (user): Подключил сервис для хеширования пароля #167

Merged
merged 1 commit into from Mar 28, 2024
Merged

feat (user): Подключил сервис для хеширования пароля #167

merged 1 commit into from Mar 28, 2024

Conversation

GhostlyIT
Copy link
Collaborator

Meta: #98078

@GhostlyIT GhostlyIT requested a review from pchelk1n March 20, 2024 19:10
@GhostlyIT GhostlyIT self-assigned this Mar 20, 2024
@GhostlyIT GhostlyIT force-pushed the 98078-feat-password-hasher branch 3 times, most recently from 0dbe9dc to 72eb9df Compare March 20, 2024 19:28
@GhostlyIT GhostlyIT force-pushed the 98078-feat-password-hasher branch 2 times, most recently from 800d4ce to 02c22fb Compare March 22, 2024 16:21
pchelk1n
pchelk1n requested changes Mar 25, 2024
View reviewed changes
Copy link
Contributor

@pchelk1n pchelk1n left a comment

Choose a reason for hiding this comment

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

Еще надо заребейзить и конфликты порешать.

backend/src/Infrastructure/HasherInterface.php Outdated Show resolved Hide resolved
backend/src/User/SignIn/Command/SignIn.php Outdated
@@ -26,6 +27,7 @@ public function __construct(
private MessageBusInterface $messageBus,
private LoggerInterface $logger,
private FindUser $findUser,
private HasherInterface $passwordHasher
Copy link
Contributor

Choose a reason for hiding this comment

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

$hasher

docker/backend/.env.dist Outdated Show resolved Hide resolved
@GhostlyIT GhostlyIT force-pushed the 98078-feat-password-hasher branch 2 times, most recently from 2b9780f to c71f3a9 Compare March 25, 2024 16:58
@GhostlyIT GhostlyIT force-pushed the 98078-feat-password-hasher branch 3 times, most recently from 9df8035 to c6c3cf6 Compare March 26, 2024 15:41
@pchelk1n pchelk1n merged commit b0b1b58 into main Mar 28, 2024
1 check passed
@pchelk1n pchelk1n deleted the 98078-feat-password-hasher branch March 28, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psprokofiev psprokofiev psprokofiev left review comments

@pchelk1n pchelk1n pchelk1n approved these changes

Assignees

@GhostlyIT GhostlyIT

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@GhostlyIT @pchelk1n @psprokofiev