Skip to content

widgets: enhance Live Log widget - #10086

Merged
swhite2 merged 4 commits into
opnsense:masterfrom
Greelan:logwidget
Jun 9, 2026
Merged

swhite2 merged 4 commits into
opnsense:masterfrom
Greelan:logwidget

Conversation

@Greelan

@Greelan Greelan commented Apr 4, 2026 •

Copy link
Copy Markdown
Contributor

Context and approach are detailed in #9684

Closes #9684

Depends on 1216d60 and 7cfb03b

@AdSchellevis AdSchellevis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Greelan the widget looks nice! I made some comments in the PR, but leave the Javascript code to review for @swhite2 next week.

Comment thread src/opnsense/mvc/app/controllers/OPNsense/Diagnostics/Api/LogController.php Outdated
Comment thread src/opnsense/www/js/widgets/LiveLog.js Outdated
AdSchellevis added a commit that referenced this pull request Apr 6, 2026
Although this isn't a full implementation yet, it can help callers that need to access the menu system.
In the long run it might be practical if this class would also construct the menu system, so we can add some flexibility there.
Uses client-side DOM for determining log endpoints, doesn't require any default target access, allows options for severity and search filter and variable log lines to show, which are displayed in summary in widget, and uses true log tailing plus backfill
Comment thread src/opnsense/www/js/widgets/Metadata/Core.xml Outdated
@Greelan Greelan closed this Apr 12, 2026
@Greelan
Greelan deleted the logwidget branch April 12, 2026 05:57
@Greelan
Greelan restored the logwidget branch April 12, 2026 09:30
@Greelan Greelan reopened this Apr 12, 2026
@Greelan

Greelan commented May 30, 2026

Copy link
Copy Markdown
Contributor Author

Any chance this will get some love?

@swhite2 swhite2 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Greelan Sorry for the wait, this slipped through on my end..

Couple of points, but the widget looks very good, thanks :)

Comment thread src/opnsense/www/js/widgets/LiveLog.js Outdated
Comment thread src/opnsense/www/js/widgets/LiveLog.js
Comment thread src/opnsense/www/js/widgets/LiveLog.js
Comment thread src/opnsense/www/js/widgets/LiveLog.js Outdated
Comment thread src/opnsense/www/js/widgets/LiveLog.js Outdated
Co-authored-by: Stephan de Wit <stephan.de.wit@deciso.com>
@swhite2
swhite2 merged commit 956a256 into opnsense:master Jun 9, 2026
@swhite2

swhite2 commented Jun 9, 2026

Copy link
Copy Markdown
Member

Merged, thanks!

@Greelan
Greelan deleted the logwidget branch June 9, 2026 20:06
fichtner pushed a commit that referenced this pull request Jun 26, 2026
Uses client-side DOM for determining log endpoints, doesn't require any default target access, allows options for severity and search filter and variable log lines to show, which are displayed in summary in widget, and uses true log tailing plus backfill

---------

Co-authored-by: Stephan de Wit <stephan.de.wit@deciso.com>
(cherry picked from commit 956a256)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Live Log widget improvements

3 participants