[PR #53] [MERGED] * Simplify vnc.php #65

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/clonos/control-pane/pull/53
Author: @bozhinov
Created: 8/2/2022
Status: ✅ Merged
Merged: 8/2/2022
Merged by: @olevole

Base: master ← Head: Public

📝 Commits (1)

32982ab * Simplify vnc.php

📊 Changes

2 files changed (+164 additions, -33 deletions)

View changed files

➕ php/validate.php (+147 -0)
📝 public/vnc.php (+17 -33)

📄 Description

add some validation of input
nodeip was not defined in all execution paths of the script
no need to check for both ip v4 and 6

_{🔄 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/53 **Author:** [@bozhinov](https://github.com/bozhinov) **Created:** 8/2/2022 **Status:** ✅ Merged **Merged:** 8/2/2022 **Merged by:** [@olevole](https://github.com/olevole) **Base:** `master` ← **Head:** `Public` --- ### 📝 Commits (1) - [`32982ab`](https://github.com/clonos/control-pane/commit/32982ab2a84e1de9991e615c9197f73a21ebae50) * Simplify vnc.php ### 📊 Changes **2 files changed** (+164 additions, -33 deletions) <details> <summary>View changed files</summary> ➕ `php/validate.php` (+147 -0) 📝 `public/vnc.php` (+17 -33) </details> ### 📄 Description - add some validation of input - nodeip was not defined in all execution paths of the script - no need to check for both ip v4 and 6 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>