Skip to content

Fix: WAF Blocking pada DataTables Menu Data → Finansial Laporan APBDes (GET ke POST)#1530

Open
habibie11 wants to merge 9 commits into
devfrom
dev-1524
Open

Fix: WAF Blocking pada DataTables Menu Data → Finansial Laporan APBDes (GET ke POST)#1530
habibie11 wants to merge 9 commits into
devfrom
dev-1524

Conversation

@habibie11
Copy link
Copy Markdown
Contributor

@habibie11 habibie11 commented May 11, 2026

issue #1524
penyesuaian api gabungan https://github.com/OpenSID/API-Database-Gabungan/pull/404

Deskripsi

Pull Request ini menyelesaikan masalah WAF (Web Application Firewall) yang memblokir request DataTables pada menu di Menu Data -> PFinansial Laporan APBDes. Pemblokiran terjadi karena request DataTables secara default menggunakan method GET, yang menghasilkan URL query string yang sangat panjang saat mengirim parameter seperti draw, columns, order, search, dan lain-lain.

Solusi yang diimplementasikan adalah mengubah metode request ajax DataTables dari GET menjadi POST, sehingga parameter dikirimkan melalui request body, bukan di URL.

Perubahan yang Dilakukan

  1. Konfigurasi DataTables (Views): Mengubah method ajax dari GET ke POST dan menambahkan header X-CSRF-TOKEN pada 4 file view:
    • data\anggaran_desa\gabungan\index.blade.php
    • data\anggaran_desa\gabungan\index.blade.php
  2. *Penyesuaian Routes (routes/web.php)

Checklist

  • Kode telah dijalankan dan dites secara lokal.
  • Fitur search, sort, dan pagination DataTables berfungsi normal dengan method POST.
  • Tidak ada error terkait CSRF Token.

@github-actions
Copy link
Copy Markdown

🔄 AI PR Review sedang antri di server...

Proses review akan segera dimulai di background — hasil akan muncul sebagai komentar setelah selesai.
Powered by CrewAI · PR #1530

@pandigresik
Copy link
Copy Markdown
Contributor

pandigresik commented May 13, 2026

Masih menggunakan get semua, posisi sinkronisasi tidak aktif

simplescreenrecorder-2026-05-13_10.21.28.mp4

yang database gabungan sudah OK, posisi sinkronisasi aktif

@affandii06
Copy link
Copy Markdown
Contributor

dibantu lanjut review mas @pandigresik

@pandigresik
Copy link
Copy Markdown
Contributor

OK, sudah sesuai

simplescreenrecorder-2026-05-13_11.28.16.mp4

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.

3 participants