Skip to content

Conversation

rvyhnal
Copy link
Contributor

@rvyhnal rvyhnal commented Sep 24, 2024

Log level selection on Monitor page - this PR fixes an issue where ALL level wouldn't return anything since the backend doesn't have the level ALL. Instead of ALL, CRITICAL is requested in the background as it returns all logs aswell.

also fixes #190

@sebix
Copy link
Member

sebix commented Sep 30, 2024

Log level selection on Monitor page - this PR fixes an issue where ALL level wouldn't return anything since the backend doesn't have the level ALL.

I'm unable to reproduce this. ALL shows all log levels for me.

Instead of ALL, CRITICAL is requested in the background as it returns all logs aswell.

CRITICAL should only include CRITICAL, ERROR should show ERROR and CRITICAL, etc. So that's a bug in intelmqctl itself.

@sorooris
Copy link

Is there anything we can do to finish the PR?

@sebix
Copy link
Member

sebix commented Jun 24, 2025

Hi @sorooris

Short answer: a 26h day :D

This problem has been lurking around for way too long. We have hit it at some demonstrations and tests, too.

The underlying issue is a bit bigger than that apparently small-looking change in the PR here
The actual solution could be certtools/intelmq#2607 (which needs some more work to do) EDIT: Problem description is in certtools/intelmq#2596

I can't promise we can get it done in June, but for July we plan for an IntelMQ release, and then a fix for the problem should be included.

@sebix
Copy link
Member

sebix commented Aug 12, 2025

The log level bug will be fixed by certtools/intelmq#2607

@sebix
Copy link
Member

sebix commented Aug 14, 2025

Also related:
#309
certtools/intelmq-api#51

I removed the workaround from this PR.

@sebix
Copy link
Member

sebix commented Aug 25, 2025

reopening for the #190 fix

@sebix sebix reopened this Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Monitor page: Automatic log refresh reset log page to first one

3 participants