Auto-split pasted URLs into scheme/host/port on forward host fields#5715
Open
Grasfer wants to merge 1 commit into
Open
Auto-split pasted URLs into scheme/host/port on forward host fields#5715Grasfer wants to merge 1 commit into
Grasfer wants to merge 1 commit into
Conversation
Pasting e.g. http://192.168.5.150:8096 into the Forward Hostname / IP field of the proxy host form (and custom locations) now fills the scheme and port fields and leaves just the host, instead of dumping the whole URL into the host field. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Author
|
As the AI mentioned i just wanted these features myself, i do not know coding at all, but it works as i wanted on my own system. |
|
Docker Image for build 1 is available on DockerHub: Note Ensure you backup your NPM instance before testing this image! Especially if there are database changes. Warning Changes and additions to DNS Providers require verification by at least 2 members of the community! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
A very common workflow when adding a proxy host is copying a service's URL straight from the browser — e.g.
http://192.168.5.150:8096— and pasting it into the Forward Hostname / IP field. Today the whole string lands in the host field, and the user has to manually delete the scheme, move the port to the port field, and set the scheme dropdown.With this change, pasting a full URL into the Forward Hostname / IP field (on the proxy host form and on custom locations) parses it and distributes the parts: the scheme dropdown and port field are filled, and only the bare hostname/IP stays in the host field. Pasting a plain hostname or IP behaves exactly as before — the split only triggers when the pasted text parses as a URL with a scheme.
No API or backend changes; frontend-only. Unit tests included for the URL parsing module (
ForwardUrl.test.ts).Full disclosure: this was AI-written (boxes checked below) — I made it because I wanted the feature myself. It's been running on my own NPM instance and works for my day-to-day use, but that's the extent of the real-world testing, so review accordingly.
Type of Change
AI Usage