Skip to content

Update bcrypt_salt_dollar.phpt#21235

Closed
l52oyl52-crypto wants to merge 1 commit intophp:masterfrom
l52oyl52-crypto:patch-3
Closed

Update bcrypt_salt_dollar.phpt#21235
l52oyl52-crypto wants to merge 1 commit intophp:masterfrom
l52oyl52-crypto:patch-3

Conversation

@l52oyl52-crypto
Copy link

@l52oyl52-crypto l52oyl52-crypto commented Feb 16, 2026

$2

$2y$13$Su6GXMjIqAAQ388E/V/65uSMxpcmCTUh4DPXzbEkxpeg7jmmvg4Bm
@l52oyl52-crypto

This comment was marked as spam.

@jordikroon
Copy link
Contributor

jordikroon commented Feb 16, 2026

May I ask what this is trying to achieve? Was this test broken?
Nevertheless the salt is pasted into the for. Which breaks syntax.

Edit. My red flags go off. New account; 2 suspicious commits (php/doc-en#5375)

@l52oyl52-crypto

This comment was marked as spam.

@iluuu1994 iluuu1994 closed this Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants