Skip to content

fix bookmarks.php#6

Open
giteku wants to merge 1 commit intomainfrom
osiris-security-patch-2
Open

fix bookmarks.php#6
giteku wants to merge 1 commit intomainfrom
osiris-security-patch-2

Conversation

@giteku
Copy link
Copy Markdown
Owner

@giteku giteku commented Jun 15, 2025

No description provided.

@giteku
Copy link
Copy Markdown
Owner Author

giteku commented Jul 5, 2025

Copilot Review:

Baris berikut:

if(!isset($_SESSION['USER_LOGGED_IN'])) {

berfungsi untuk memeriksa apakah user sudah login atau belum. Dengan menghapus baris ini, pesan "Please Login to See Your Bookmarks" akan selalu muncul, bahkan untuk user yang sudah login. Hal ini bisa menyebabkan pengalaman pengguna yang kurang baik.

Saran:
Sebaiknya bagian pengecekan login tetap dipertahankan agar alert dan redirect hanya terjadi untuk user yang belum login. Jika autentikasi sudah ditangani di tempat lain, mohon sertakan dokumentasinya agar alur autentikasi jelas bagi pengembang lain.

@giteku giteku closed this Jul 5, 2025
@giteku giteku reopened this Jul 5, 2025
@giteku giteku reopened this Jul 5, 2025
@giteku giteku reopened this Jul 5, 2025
@giteku giteku reopened this Jul 5, 2025
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