Skip to content

feat(support): add technical support information page#7174

Open
somethingwithproof wants to merge 3 commits into
Cacti:1.2.xfrom
somethingwithproof:feat/1.2.x-support-page
Open

feat(support): add technical support information page#7174
somethingwithproof wants to merge 3 commits into
Cacti:1.2.xfrom
somethingwithproof:feat/1.2.x-support-page

Conversation

@somethingwithproof
Copy link
Copy Markdown
Contributor

@somethingwithproof somethingwithproof commented May 29, 2026

Adds support.php, an authenticated diagnostics page that summarises rrdtool/snmp versions, recent database queries, and running background processes for support triage.

Carved out of #7172, which bundled it with unrelated changes; this is the standalone page so it can be reviewed on its own. Adds a filter_sanitize() wrapper on the table filter for the page's sanitize-only path.

Base: 1.2.x. Page requires authentication.

Copilot AI review requested due to automatic review settings May 29, 2026 15:49
@TheWitness
Copy link
Copy Markdown
Member

Merge conflicts @somethingwithproof, can you show a screen shot of the example support page?

Adds support.php, an authenticated diagnostics page (rrdtool/snmp versions,
recent database queries, running background processes), plus a filter_sanitize()
wrapper on the table filter for the page's sanitize-only path.

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
@somethingwithproof somethingwithproof force-pushed the feat/1.2.x-support-page branch from d81e395 to ce277d7 Compare June 7, 2026 08:41
@somethingwithproof
Copy link
Copy Markdown
Contributor Author

Rebased onto current 1.2.x, the conflicts are cleared. Screenshot of the support page to follow.

Rebase onto 1.2.x shifted two dynamic_include line numbers in lib/rrd.php;
regenerate the tracked sink inventory so the guard passes.

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants