github-starred/pangolin
2
0
Fork 0
mirror of https://github.com/fosrl/pangolin.git synced 2026-05-07 13:19:07 -05:00
Code Issues 564 Packages Projects Releases 72 Wiki Activity

Unable to set site remote subnets #740

New Issue
Closed
opened 2025-11-13 12:09:49 -06:00 by GiteaMirror · 1 comment
GiteaMirror commented 2025-11-13 12:09:49 -06:00
Owner
Copy Link

Originally created by @aarononeal on GitHub (Oct 21, 2025).

Describe the Bug

When editing a Site, no matter what you enter into the CIDR box, the API call POSTs an empty remoteSubnets property.

{
  "name": "DMZ",
  "dockerSocketEnabled": true,
  "remoteSubnets": ""
}

Which of course just returns success and then reloading the page shows the CIDR did not persist.

This makes it impossible to establish routing between sites.

Environment

  • OS Type & Version: Ubuntu 24.04 LTS
  • Pangolin Version: 1.11.1
  • Gerbil Version: 1.2.2
  • Traefik Version: 3.5.3
  • Newt Version: 1.5.2
  • Olm Version: N/A

To Reproduce

  1. Create a new Site
  2. Edit the Site
  3. Enter CIDR into Remote Subnets
  4. Save All Settings
  5. Monitor POST using browser tools and note remoteSubnets = "" or reload page and note CIDR is empty

Expected Behavior

The CIDR should be persisted and routes added to Gerbil.

Originally created by @aarononeal on GitHub (Oct 21, 2025). ### Describe the Bug When editing a Site, no matter what you enter into the CIDR box, the API call POSTs an empty `remoteSubnets` property. ```json { "name": "DMZ", "dockerSocketEnabled": true, "remoteSubnets": "" } ``` Which of course just returns success and then reloading the page shows the CIDR did not persist. This makes it impossible to establish routing between sites. ### Environment - OS Type & Version: Ubuntu 24.04 LTS - Pangolin Version: 1.11.1 - Gerbil Version: 1.2.2 - Traefik Version: 3.5.3 - Newt Version: 1.5.2 - Olm Version: N/A ### To Reproduce 1. Create a new Site 2. Edit the Site 3. Enter CIDR into Remote Subnets 4. Save All Settings 5. Monitor POST using browser tools and note `remoteSubnets` = `""` or reload page and note CIDR is empty ### Expected Behavior The CIDR should be persisted and routes added to Gerbil.
GiteaMirror commented 2025-11-13 12:09:49 -06:00
Author
Owner
Copy Link

@aarononeal commented on GitHub (Oct 21, 2025):

Resolved: You can't just click Save. You have to hit enter in the CIDR field before Save.

Might be helpful to add a note indicating that field doesn't work the same as the others (e.g. press enter to parse the CIDR).

@aarononeal commented on GitHub (Oct 21, 2025): Resolved: You can't just click Save. You have to hit enter in the CIDR field before Save. Might be helpful to add a note indicating that field doesn't work the same as the others (e.g. press enter to parse the CIDR).
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
wontfix
No Label
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#740
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?