github-starred/pangolin
2
0
Fork 0
mirror of https://github.com/fosrl/pangolin.git synced 2026-03-16 20:00:21 -05:00
Code Issues 94 Packages Projects Releases 48 Wiki Activity

[PR #1316] [MERGED] Enhancement #906/Smart Host Parsing for Target Address #1030

New Issue
Closed
opened 2025-11-13 12:16:31 -06:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-13 12:16:31 -06:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/1316
Author: @Pallavikumarimdb
Created: 8/22/2025
Status: ✅ Merged
Merged: 8/30/2025
Merged by: @oschwartz10612

Base: dev ← Head: feature-906/smart-host-parsing

📝 Commits (2)

📊 Changes

5 files changed (+209 additions, -154 deletions)

View changed files

📝 server/lib/remoteProxy.ts (+1 -1)
📝 server/routers/gerbil/getConfig.ts (+1 -1)
📝 src/app/[orgId]/settings/resources/[resourceId]/proxy/page.tsx (+93 -69)
📝 src/app/[orgId]/settings/resources/create/page.tsx (+99 -83)
➕ src/lib/parseHostTarget.ts (+15 -0)

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

This PR introduces smart host parsing for the Target Address field (issues highlighted in #906 ).
Sub-issues addressed: 2.1 Smart Parsing for Host Targets

Solution:
Users can now paste a full URL (e.g., http://192.168.1.2:81) directly into the IP/Hostname input, and the system will automatically extract and populate method, host, and port fields. Users still have the flexibility to manually edit each field if required.

  • Added a parseHostTarget utility to extract protocol, host, and port from a pasted/typed URL or raw IP input.
  • Auto-assign default ports when user input does not include one:
    • http:// → port 80
    • https:// → port 443
  • Preserves user-provided ports if specified.
  • Works for both domain names and IP addresses.

How to test?

🔄 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/fosrl/pangolin/pull/1316 **Author:** [@Pallavikumarimdb](https://github.com/Pallavikumarimdb) **Created:** 8/22/2025 **Status:** ✅ Merged **Merged:** 8/30/2025 **Merged by:** [@oschwartz10612](https://github.com/oschwartz10612) **Base:** `dev` ← **Head:** `feature-906/smart-host-parsing` --- ### 📝 Commits (2) - [`9557f75`](https://github.com/fosrl/pangolin/commit/9557f755a5b0fca33e7ffe41d452a9e62bec2e37) Add Smart Host Parsing - [`fb1481c`](https://github.com/fosrl/pangolin/commit/fb1481c69cc07b2b5420bf177e526906199fc9b2) fix lint issue ### 📊 Changes **5 files changed** (+209 additions, -154 deletions) <details> <summary>View changed files</summary> 📝 `server/lib/remoteProxy.ts` (+1 -1) 📝 `server/routers/gerbil/getConfig.ts` (+1 -1) 📝 `src/app/[orgId]/settings/resources/[resourceId]/proxy/page.tsx` (+93 -69) 📝 `src/app/[orgId]/settings/resources/create/page.tsx` (+99 -83) ➕ `src/lib/parseHostTarget.ts` (+15 -0) </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description This PR introduces smart host parsing for the Target Address field (issues highlighted in #906 ). **Sub-issues addressed: 2.1 Smart Parsing for Host Targets** **Solution:** Users can now paste a full URL (e.g., http://192.168.1.2:81) directly into the IP/Hostname input, and the system will automatically extract and populate method, host, and port fields. Users still have the flexibility to manually edit each field if required. * Added a parseHostTarget utility to extract protocol, host, and port from a pasted/typed URL or raw IP input. * Auto-assign default ports when user input does not include one: - http:// → port 80 - https:// → port 443 * Preserves user-provided ports if specified. * Works for both domain names and IP addresses. ## How to test? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-13 12:16:31 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
api
authentication
bug
config
dependencies
docker
documentation
enhancement
good first issue
help wanted
Improvement
Look Into
needs investigating
networking
new feature
non-critical bug
potential bug
pull-request
Mirrored from GitHub Pull Request
 question
reverse proxy
Security
stale
ui
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#1030
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?