Skip to content

Fix: WAF Blocking di datatables dengan ubah GET ke POST di Menu Data -> Kesehatan#1533

Open
habibie11 wants to merge 22 commits into
devfrom
dev-1521
Open

Fix: WAF Blocking di datatables dengan ubah GET ke POST di Menu Data -> Kesehatan#1533
habibie11 wants to merge 22 commits into
devfrom
dev-1521

Conversation

@habibie11
Copy link
Copy Markdown
Contributor

issue #1521

Summary

  • ubah endpoint DataTables pada submenu kesehatan AKI & AKB, Imunisasi, dan Epidemi Penyakit menjadi POST only untuk menghindari query string panjang yang berpotensi diblokir WAF
  • sesuaikan konfigurasi AJAX DataTables pada tiga halaman kesehatan agar mengirim request dengan method POST
  • tambahkan cakupan test untuk memastikan endpoint kesehatan menerima POST, mengembalikan struktur DataTables yang benar, dan menolak GET

Testing

  • php artisan test tests/Feature/Kesehatan/AkiAkbCrudTest.php tests/Feature/Kesehatan/ImunisasiCrudTest.php tests/Feature/Kesehatan/KesehatanDataTablesPostTest.php

@habibie11 habibie11 requested a review from affandii06 May 12, 2026 04:40
@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 #1533

@pandigresik
Copy link
Copy Markdown
Contributor

Masih ada yang menggunakan get /data/toilet-sanitasi

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

@affandii06
Copy link
Copy Markdown
Contributor

dibantu perbaiki mas @habibie11

@habibie11
Copy link
Copy Markdown
Contributor Author

Masih ada yang menggunakan get /data/toilet-sanitasi

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

menu toilet & sanitasi tidak disebutkan dalam issue, apakah diterapkan juga mas @affandii06 ?

@affandii06
Copy link
Copy Markdown
Contributor

oh iya, tidak mas @habibie11 , selain menu toilet & sanitasi aman ya mas @pandigresik ?

@pandigresik
Copy link
Copy Markdown
Contributor

oh iya, tidak mas @habibie11 , selain menu toilet & sanitasi aman ya mas @pandigresik ?

aman

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