Ilia Ross

Fonts' contrast must fit accessibility standards:  For some color palettes it's a bit less though but still close, and well visible? Is this how you see it?  >...

Oh, wow. This is an interesting issue. I have been looking around it for hours trying to figure out the solution. I think I [fixed it locally](https://github.com/webmin/webmin/commit/3c687b46be545e7489bfe5848c960cca57529ee9) but it's not...

I am endlessly confused! It appears that the fix I just made above is the only possible right solution? The actual breakage was happening at the line I fixed (the...

Yes, I didn't modify it. Yes, I quoted it in the caller.

Should we not allow a user to have **`tempdir`** with spaces? I know it's super rare but still.

Okay, great. I don't mind as longs as it works safe and fine.. I think we need to audit other modules, using regex search or something to try to locate...

> Best practice in Webmin is to always use quotameta on paths, as failing to do so can cause security issues if the path is in any way controlled by...

Agreed! Thank you for pointing this out in the first place.

> I think we need to audit other modules, using regex search or something to try to locate familiar case senarious. For example, when **`tempdir`** set with spaces, in Virtualmin...

Thanks. But script installation still fails when `tmpdir` contains spaces :slightly_smiling_face: