Skip to content

Respect log level setting by client #868

@cliffhall

Description

@cliffhall

Is your feature request related to a problem? Please describe.
This issue is paired with this request in the Inspector project:

And arises from a recent set of changes...

...that let log messages surface in the UI. For testing, the Everything example server was also amended to periodically send randomly leveled log messages. Now we want to add log level throttling.

Describe the solution you'd like
Right now, the setLevelRequest handler in the Everything server doesn't persist the level, it just sends a dummy debug log message indicating the new level was set.

We need to track the level, with a default set the same as the client, highest level letting all messages through.

In the setInterval handler that sends dummy log messages periodically, we should compare the level of the outgoing message and only send if appropriate.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Not doing this at all. Seems like we should though.

Additional context
Add any other context or screenshots about the feature request here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions