Skip to content

Pre-release 4.2.106 → 4.2.105 — task 4.3 hashing swap (illuminate/hashing v13) - #77

Open
agissept wants to merge 1 commit into
pre-release/4.2.105from
pre-release/4.2.106
Open

agissept wants to merge 1 commit into
pre-release/4.2.105from
pre-release/4.2.106

Conversation

@agissept

Copy link
Copy Markdown
Member

Task 4.3 (L2 per-unit): swap illuminate/hashing → ^13 (done-class no-op port). Stacked di atas #76 (4.2.105 translation).

Perubahan

  • composer: illuminate/hashing replace → require ^13 (v13.33.0). v13 Hashing\* shadow tree fork (PSR-4 seed).
  • L4.2 & v13 sama-sama native password_* + PASSWORD_BCRYPT → hash tersimpan tetap valid (cost default 10→12; hash cost-10 lama tetap verify, tak ada lockout). 'hash' kini bind ke HashManager (+ Contracts\Hashing\Hasher).
  • Foundation\Application: buang alias legacy mati 'hash'→Illuminate\Hashing\HasherInterface (BC task 2.11; irrelevant setelah hashing v13, 0 consumer). Alias contract dipertahankan.
  • FoundationApplicationTest: assert via contract saja.

Fork suite 527 hijau; roundtrip + verify hash cost-10 lama terkonfirmasi.

App matched-pair: dicoding#5915.

'done'-class no-op port: both L4.2 and v13 sit on native password_* with PASSWORD_BCRYPT,
so stored hashes stay verifiable across the flip (default cost 10 -> 12; existing cost-10
hashes still verify, no lockout). v13 'hash' binds a HashManager (multi-driver) and ships
Contracts\Hashing\Hasher; v13 Hashing\* shadows the fork tree via the PSR-4 seed mechanism.

- composer: illuminate/hashing self.version -> ^13 (v13.33.0).
- Foundation\Application: drop the now-dead legacy alias 'hash' -> Illuminate\Hashing\
  HasherInterface (task 2.11 BC bridge; the fork interface is irrelevant once hashing is v13,
  zero consumers). Contract alias 'hash' -> Contracts\Hashing\Hasher kept.
- FoundationApplicationTest: retarget the alias test to the contract only (drop the legacy
  HasherInterface assertion).

Fork suite 527 green; make/check roundtrip + legacy cost-10 verify confirmed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant