Bug #13413
closedSome messages presented to users contain relative links to pages which may be invalid when triggered from certain packages
100%
Description
If something goes wrong when you save the config changes of Wireguard (can't determine what it was in my case)
you will get the message:
"There was a problem applying the changes. See the System Logs"
The "System logs" link goes to the 404 page and usr/local/www/wg/status_logs.php is not exist.
Files
Updated by Kris Phillips over 2 years ago
I can't reproduce this, but only because I can't get this error to pop up. I've tried intentionally breaking Wireguard to get it to do this and I'm not sure how to recreate it.
However, the URL for wireguard's configuration uses https://[IP of firewall]/wg/[insert php file here]. As such, any links to logging that use the global functions would try to tack in the /wg/ part, which is probably the core of the issue here. Probably need to update the PHP code here to not use the global functions and instead use something else.
Updated by aleksei prokofiev about 1 year ago
Tested on
23.09-RELEASE (amd64)
built on Fri Nov 17 13:32:00 UTC 2023
FreeBSD 14.0-CURRENT
I've tried different setting to try get this error, but can't reproduce.
Updated by Christopher Cope 12 months ago
- Status changed from New to Pull Request Review
- Assignee set to Christopher Cope
https://gitlab.netgate.com/pfSense/pfSense/-/merge_requests/1111
I wasn't able to reproduce the error with Wireguard, but the functions for the error message use relative links, so I made a merge request to point them at the root domain.
Updated by Jim Pingle 12 months ago
- Project changed from pfSense Packages to pfSense
- Subject changed from link to not existed /wg/status_logs.php to Some messages presented to users contain relative links to pages which may be invalid when triggered from certain packages
- Category changed from WireGuard to Web Interface
- Target version set to 2.8.0
- Plus Target Version set to 24.03
- Release Notes set to Default
Moving projects/rewording since this isn't specific to any one package, it just happens to affect packages which put files in subdirectories.
Updated by Christopher Cope 11 months ago
- Status changed from Pull Request Review to Feedback
- % Done changed from 0 to 100
Applied in changeset b505dae3a3accf4757d3b617e91a60f987d35841.