Update to 23.1.5 & Service status panel visual glitch

Started by mg85, March 29, 2023, 03:08:12 PM

Previous topic - Next topic
Hi,

I have just updated to 23.1.5, and found out that the dashboard is no longer populating the daemon names for udpbroadcastrelay. On my firewall, I have two configured: One for mDNS and one for SSDP.
They are listed under the Services panel on the dashboard.
I now notice that:

- The name for the SSDP daemon is blank
- The mDNS daemon is not at all visible (but is started and does work, since I can connect to devices over mdns)

See screenshots for some more details.
I expect this to be a bug in the latest release. Hopefully someone can jump on this at a convenient time.
Functionality is working, this is just a visual glitch.
@Franco let me know in case you need some further logs to troubleshoot :)

Thats a bit odd, but let's see what this returns:

# pluginctl -S udpbroadcastrelay

(new feature also useful for debugging)


Cheers,
Franco

At first glance this would probably help:

https://github.com/opnsense/plugins/commit/c08a2ea17

# opnsense-patch -c plugins c08a2ea17


Cheers,
Franco

Hi Franco, thanks a lot for your swift response! Here's the output of the command:

[
    {
        "description": {},
        "id": {},
        "pidfile": "/var/run/udpbroadcastrelay_1.pid",
        "configd": {
            "restart": [
                "udpbroadcastrelay restart 1"
            ],
            "start": [
                "udpbroadcastrelay start 1"
            ],
            "stop": [
                "udpbroadcastrelay stop 1"
            ]
        },
        "name": "udpbroadcastrelay",
        "status": "udpbroadcastrelay[1] is running as pid 21360."
    },
    {
        "description": {},
        "id": {},
        "pidfile": "/var/run/udpbroadcastrelay_2.pid",
        "configd": {
            "restart": [
                "udpbroadcastrelay restart 2"
            ],
            "start": [
                "udpbroadcastrelay start 2"
            ],
            "stop": [
                "udpbroadcastrelay stop 2"
            ]
        },
        "name": "udpbroadcastrelay",
        "status": "udpbroadcastrelay[2] is running as pid 22969."
    }
]


The patch fixed it :) - thanks for your hard work!

Yay, good. Did you update from 22.7? It looks more like a PHP-side turn of events.

But code was not correct for a long time in the plugin either way.


Cheers,
Franco

Or... funnelling the info through JSON encode/decode makes it harder for PHP to guess if it should return an object or string... :) So you were on 23.1.4 then.


Cheers,
Franco

Quote from: franco on March 29, 2023, 03:35:03 PM
Or... funnelling the info through JSON encode/decode makes it harder for PHP to guess if it should return an object or string... :) So you were on 23.1.4 then.


Cheers,
Franco

Yep - I was on 23.1.4 :)
I'm glad that you have found a solution so quickly. I guess this somehow will roll up into 23.1.6?