Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

No logs accepted if global blacklist is empty #23

Closed
mcm1957 opened this issue Feb 21, 2023 · 1 comment
Closed

No logs accepted if global blacklist is empty #23

mcm1957 opened this issue Feb 21, 2023 · 1 comment
Labels
bug Something isn't working fixed
Milestone
Release 2.1.x

Comments

@mcm1957
Copy link
Member

mcm1957 commented Feb 21, 2023

Original: Mic-M/ioBroker.logparser#35

Describe the bug
When disableing all entries at the global blacklist by unchecking them, log-parser does not accept (register) ANY new logs

To Reproduce
Steps to reproduce the behavior:

  1. Install a fresh instance of log-parser
  2. Do not change anything - just start with the default configuration
  3. cause any log to be written, i.e. by restarting an (other) adapter instance
  4. The event should be visible at "info" json object
  5. open configuration of log-parser, go to tab "global blacklist" and deselect all entries. (Do not delete them, just deselect them.)
  6. restart log-parser
  7. cause any log to be written like at 3.
  8. Problem: logparser deos no longer add this event to the corresponding json object

Expected behavior
An empty global blacklist should not cause any problems.

Versions:

  • Adapter version: 1.1.0
  • JS-Controller version: 4.0.23
  • Node version: 14.19.3
  • Operating system: Raspian
@mcm1957 mcm1957 added the bug Something isn't working label Feb 21, 2023
ciddi89 added a commit that referenced this issue Mar 5, 2023
@ciddi89
fixes: No logs accepted if global blacklist is empty #23
eb9de9e
ciddi89 added a commit that referenced this issue Mar 5, 2023
@ciddi89
chore: release v2.1.0
74760dd 
-   (ciddi89) Added: [#24](#24) Option to remove 'COMPACT' in log entries
-   (ciddi89) Added: [#21](#21) Option to remove only 'script.js' in log entries
-   (ciddi89) Fixed: [#46](#46) Midnight function to change today/yesterday
-   (ciddi89) Fixed: [#23](#23) When nothing selected in blacklist, adapter didn't work anymore
-   (ciddi89) Other: Small code and translation improvements
@ciddi89
Copy link
Contributor

ciddi89 commented Mar 5, 2023

Fixed in 2.1.0

@ciddi89 ciddi89 closed this as completed Mar 5, 2023
@mcm1957 mcm1957 removed the in Test label Mar 5, 2023
@mcm1957 mcm1957 added this to the Release 2.1.x milestone Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working fixed
Projects
None yet
Milestone
Release 2.1.x
Development

No branches or pull requests
2 participants
@mcm1957 @ciddi89