[PR #491] [MERGED] Fix PORT_FLOOR check and CIDR_MAX_BITS in AssignmentService #787

Closed
opened 2026-02-04 18:50:42 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/491
Author: @Boy132
Created: 7/17/2024
Status: Merged
Merged: 7/17/2024
Merged by: @Boy132

Base: mainHead: fix/port-range


📝 Commits (2)

📊 Changes

2 files changed (+4 additions, -4 deletions)

View changed files

📝 app/Services/Allocations/AssignmentService.php (+3 -3)
📝 lang/en/exceptions.php (+1 -1)

📄 Description

Ports needs to be "greater than or equal to 1024". 1024 itself works fine:
image

Also fixes the max CIDR mask, ref https://github.com/pterodactyl/panel/pull/5111.


🔄 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/pelican-dev/panel/pull/491 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 7/17/2024 **Status:** ✅ Merged **Merged:** 7/17/2024 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `fix/port-range` --- ### 📝 Commits (2) - [`72f2114`](https://github.com/pelican-dev/panel/commit/72f2114408b01cae48e10ba20805fe12357bd0cb) fix max cidr - [`b368ebb`](https://github.com/pelican-dev/panel/commit/b368ebb54c0785cda32291ac1814838e14163e5b) fix port floor ### 📊 Changes **2 files changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `app/Services/Allocations/AssignmentService.php` (+3 -3) 📝 `lang/en/exceptions.php` (+1 -1) </details> ### 📄 Description Ports needs to be "greater than or equal to 1024". 1024 itself works fine: ![image](https://github.com/user-attachments/assets/07b39184-26c7-43aa-9671-a419b8deb50b) Also fixes the max CIDR mask, ref https://github.com/pterodactyl/panel/pull/5111. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 18:50:42 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#787