[PR #20] [MERGED] Fixed most SQL Injections #50

New issue

Closed

opened 2026-03-02 03:21:32 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 03:21:32 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/clonos/control-pane/pull/20
Author: @Stefar77
Created: 4/24/2019
Status: ✅ Merged
Merged: 4/30/2019
Merged by: @olevole

Base: master ← Head: master

📝 Commits (8)

d411ee4 Fixed most SQL Injections
67e7ba3 Escapes and code a bit cleaner.
07b7c0c Style updates and small fixes
717c839 One more SQL Injection fixed
8c82d3e Less concatting in queries to make it cleaner
7860bcb Style fixes en injection todo's
f5fd0c3 SQL commands upper looks cleaner
19aad66 Fixed 'WHER' typo in query

📊 Changes

2 files changed (+783 additions, -1237 deletions)

View changed files

📝 php/clonos.php (+735 -1157)
📝 php/db.php (+48 -80)

📄 Description

Did a few fixes yesterday.
Still needs more fixes (like filtering for shell injections) and some more tweaks, working on that tonight.
I may split up the clonos object into separate objects (i.e. jail, bhyve, node) next update.

Also still working on the formatting a bit, it's a pain to read when single commands span multiple lines without exceeding low-res text screen lengths. (~80 cols)

_{🔄 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/clonos/control-pane/pull/20 **Author:** [@Stefar77](https://github.com/Stefar77) **Created:** 4/24/2019 **Status:** ✅ Merged **Merged:** 4/30/2019 **Merged by:** [@olevole](https://github.com/olevole) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (8) - [`d411ee4`](https://github.com/clonos/control-pane/commit/d411ee4b6b1272f87bd6745292172dd75dadd2f1) Fixed most SQL Injections - [`67e7ba3`](https://github.com/clonos/control-pane/commit/67e7ba33bbcb9e6a8acfdecb01873f371e691ad4) Escapes and code a bit cleaner. - [`07b7c0c`](https://github.com/clonos/control-pane/commit/07b7c0c2363e5b836eb6bcefca116389b72770f3) Style updates and small fixes - [`717c839`](https://github.com/clonos/control-pane/commit/717c839d521a1485775f1fbc8e0adcaaee58bfe2) One more SQL Injection fixed - [`8c82d3e`](https://github.com/clonos/control-pane/commit/8c82d3eab4ca2cf50a9047fa4ec5850f39a67205) Less concatting in queries to make it cleaner - [`7860bcb`](https://github.com/clonos/control-pane/commit/7860bcba09fb5cd726c4bea9766fb5388980d127) Style fixes en injection todo's - [`f5fd0c3`](https://github.com/clonos/control-pane/commit/f5fd0c3856511b7ce96e8a7e1feaf433f5f00df8) SQL commands upper looks cleaner - [`19aad66`](https://github.com/clonos/control-pane/commit/19aad6636b25fa6bd467570995da6df6636b00fd) Fixed 'WHER' typo in query ### 📊 Changes **2 files changed** (+783 additions, -1237 deletions) <details> <summary>View changed files</summary> 📝 `php/clonos.php` (+735 -1157) 📝 `php/db.php` (+48 -80) </details> ### 📄 Description Did a few fixes yesterday. Still needs more fixes (like filtering for shell injections) and some more tweaks, working on that tonight. I may split up the clonos object into separate objects (i.e. jail, bhyve, node) next update. * Also still working on the formatting a bit, it's a pain to read when single commands span multiple lines without exceeding low-res text screen lengths. (~80 cols) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-02 03:21:32 +03:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/control-pane#50

No description provided.

Rows
Columns