Skip to content

Migrasi autentikasi dari Bearer Token ke Sanctum Cookie-based - #17

Merged
spectraVal merged 2 commits into
developmentfrom
FE
Jun 23, 2026
Merged

Migrasi autentikasi dari Bearer Token ke Sanctum Cookie-based#17
spectraVal merged 2 commits into
developmentfrom
FE

Conversation

@spectraVal

@spectraVal spectraVal commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

Revisi implementasi autentikasi dari Bearer Token ke Sanctum Cookie-based Session (SPA mode) sesuai ADR-006 dan OpenAPI contract v2.2 yang telah diupdate oleh BE. Perubahan ini adalah prerequisite sebelum AUTH-03 dan AUTH-04 bisa dikerjakan.

Jira Ticket

Ticket: -

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation update
  • Configuration change

Changes Made

  • src/types/auth.ts — hapus field token dan token_type dari LoginResponseData dan AuthState
  • src/api/axiosInstance.ts — hapus interceptor Bearer Token; auth sepenuhnya dihandle via cookie dan withCredentials: true
  • src/services/authService.ts — tambah initCsrf() yang hit GET /sanctum/csrf-cookie sebelum register dan login; tambah VITE_API_BASE_URL untuk base URL non-/api
  • src/contexts/AuthContext.tsx — hapus token state; hanya simpan user object; isAuthenticated derived dari user !== null
  • src/tests/pages/RegisterPage.test.tsx — sesuaikan mock context dan payload assertion dengan contract v2.2
  • src/tests/pages/LoginPage.test.tsx — sesuaikan mock context dan mockLoginResponse dengan contract v2.2
  • .env.example — tambah VITE_API_BASE_URL

How to Test

  1. Copy .env.example ke .env.local, set VITE_API_URL dan VITE_API_BASE_URL
  2. Pastikan BE running di URL yang sesuai
  3. Buka /register, isi form, submit — verifikasi redirect ke /login dengan success message
  4. Di /login, login dengan credentials valid — verifikasi redirect ke /dashboard
  5. Buka DevTools → Application → Cookies — verifikasi laravel_session dan XSRF-TOKEN ter-set setelah login
  6. Jalankan unit test: npm run test

Screenshots (if applicable)

Checklist

  • Kode sudah di-test secara lokal
  • Tidak ada console.log yang tertinggal
  • Tidak ada conflict dengan branch target
  • Nama branch mengikuti konvensi (feature/QNH-XXX-...)
  • PR title mengikuti konvensi (QNH-XXX | Deskripsi singkat)

@spectraVal
spectraVal merged commit ce54820 into development Jun 23, 2026
3 checks passed
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