enkore / i3pystatus

A complete replacement for i3status
https://i3pystatus.readthedocs.io/
MIT License
445 stars 188 forks source link

Empty color string floods ~/.xsession-errors #863

Closed elig0n closed 5 months ago

elig0n commented 7 months ago

My disk module had a color="" line which caused a flooding of such messages to the ~/.xsession-errors file (which reached tens of GiBs in some time)

[libi3] ERROR: Could not parse color:

My suggestion is for you to consider adding a check that will replace an empty passed color with the default one.

terminalmage commented 6 months ago

@elig0n can you test the changes in https://github.com/enkore/i3pystatus/pull/864?

elig0n commented 6 months ago

@elig0n can you test the changes in #864?

Works for me

terminalmage commented 5 months ago

Thanks, merged that PR. Sorry for the delay.