[PR #576] [MERGED] fix: update error handling and security domain checks #1195

New issue

Closed

opened 2026-03-15 15:57:44 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 15:57:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/flyimg/flyimg/pull/576
Author: @sadok-f
Created: 9/4/2025
Status: ✅ Merged
Merged: 9/5/2025
Merged by: @sadok-f

Base: main ← Head: fix-allowed-domains

📝 Commits (10+)

dca76f5 fix: update error handling and security domain checks
a31cd1d refactor: clean up ErrorHandler formatting and improve readability
4489f9b refactor: enhance exception.js for improved readability and functionality, including better tab and toggle handling
4ebbc26 refactor: streamline exception.js with ES6 syntax and enhance class handling for improved maintainability
ff714be fix: update exception message for domain restriction in SecurityHandler
4e70707 fix linting errors
f265673 refactor: improve formatting and readability
d745698 refactor: improve code formatting and readability in ErrorHandler
4770173 ensure docker removal command does not fail if container is not running
706aa37 update Docker run command to use port 9900 for development deployment

📊 Changes

40 files changed (+1946 additions, -121 deletions)

View changed files

📝 .github/workflows/branch-dev-deploy.yml (+2 -2)
📝 README.md (+6 -12)
📝 app.php (+2 -6)
📝 config/parameters.yml (+4 -0)
📝 docs/basic-usage.md (+29 -29)
📝 docs/configuration.md (+17 -1)
📝 docs/helm-chart.md (+11 -0)
📝 docs/index.md (+15 -11)
📝 docs/install.md (+1 -1)
📝 docs/security.md (+9 -2)
📝 docs/url-options.md (+39 -39)
📝 src/Core/Entity/Response.php (+0 -3)
➕ src/Core/Handler/ErrorHandler.php (+336 -0)
📝 src/Core/Handler/SecurityHandler.php (+47 -7)
📝 src/Core/Views/Default/index.html (+2 -2)
➕ src/Core/Views/Errors/assets/css/error.css (+4 -0)
➕ src/Core/Views/Errors/assets/css/exception.css (+359 -0)
➕ src/Core/Views/Errors/assets/css/exception_full.css (+128 -0)
➕ src/Core/Views/Errors/assets/images/chevron-right.svg (+1 -0)
➕ src/Core/Views/Errors/assets/images/favicon.png.base64 (+1 -0)

...and 20 more files

📄 Description

fix: update error handling and security domain checks

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/flyimg/flyimg/pull/576 **Author:** [@sadok-f](https://github.com/sadok-f) **Created:** 9/4/2025 **Status:** ✅ Merged **Merged:** 9/5/2025 **Merged by:** [@sadok-f](https://github.com/sadok-f) **Base:** `main` ← **Head:** `fix-allowed-domains` --- ### 📝 Commits (10+) - [`dca76f5`](https://github.com/flyimg/flyimg/commit/dca76f5f6eacff7d0c187a6733c3130c56550d61) fix: update error handling and security domain checks - [`a31cd1d`](https://github.com/flyimg/flyimg/commit/a31cd1d4f2f39103ff4424f345118c9e6305e426) refactor: clean up ErrorHandler formatting and improve readability - [`4489f9b`](https://github.com/flyimg/flyimg/commit/4489f9b664edcde1305f8fc2b0a4b5671d7caf25) refactor: enhance exception.js for improved readability and functionality, including better tab and toggle handling - [`4ebbc26`](https://github.com/flyimg/flyimg/commit/4ebbc260d3200eb045357c7027e27e0371a2473d) refactor: streamline exception.js with ES6 syntax and enhance class handling for improved maintainability - [`ff714be`](https://github.com/flyimg/flyimg/commit/ff714be010a1bf604e51834b43be70a902f512fd) fix: update exception message for domain restriction in SecurityHandler - [`4e70707`](https://github.com/flyimg/flyimg/commit/4e707078c1b6979453194ce65f7a3eb131524f19) fix linting errors - [`f265673`](https://github.com/flyimg/flyimg/commit/f26567344299a6277166416408acc5af13f57a6e) refactor: improve formatting and readability - [`d745698`](https://github.com/flyimg/flyimg/commit/d745698e775b18faf7fede65ca8dca1838abe5d9) refactor: improve code formatting and readability in ErrorHandler - [`4770173`](https://github.com/flyimg/flyimg/commit/4770173d060f964bf05715ce30a4cbe306373eb5) ensure docker removal command does not fail if container is not running - [`706aa37`](https://github.com/flyimg/flyimg/commit/706aa3717583c339a94c08a278430d54093b61cc) update Docker run command to use port 9900 for development deployment ### 📊 Changes **40 files changed** (+1946 additions, -121 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/branch-dev-deploy.yml` (+2 -2) 📝 `README.md` (+6 -12) 📝 `app.php` (+2 -6) 📝 `config/parameters.yml` (+4 -0) 📝 `docs/basic-usage.md` (+29 -29) 📝 `docs/configuration.md` (+17 -1) 📝 `docs/helm-chart.md` (+11 -0) 📝 `docs/index.md` (+15 -11) 📝 `docs/install.md` (+1 -1) 📝 `docs/security.md` (+9 -2) 📝 `docs/url-options.md` (+39 -39) 📝 `src/Core/Entity/Response.php` (+0 -3) ➕ `src/Core/Handler/ErrorHandler.php` (+336 -0) 📝 `src/Core/Handler/SecurityHandler.php` (+47 -7) 📝 `src/Core/Views/Default/index.html` (+2 -2) ➕ `src/Core/Views/Errors/assets/css/error.css` (+4 -0) ➕ `src/Core/Views/Errors/assets/css/exception.css` (+359 -0) ➕ `src/Core/Views/Errors/assets/css/exception_full.css` (+128 -0) ➕ `src/Core/Views/Errors/assets/images/chevron-right.svg` (+1 -0) ➕ `src/Core/Views/Errors/assets/images/favicon.png.base64` (+1 -0) _...and 20 more files_ </details> ### 📄 Description fix: update error handling and security domain checks --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>