Icon import bugs and a suggestion to obfuscate imported filename

[t214c]

Have you searched for an existing issue?

• Yes, I tried searching and reviewed the pinned issues

Brief Summary

See attached icons:

1 is rendered black.
2 is declared with "small" dimensions in SVG but it still an SVG quality. KeePassXC imports it very blurry. I think it should be scaled first then imported.

Steps to Reproduce

icons.zip

Operating System

Windows

v 2.7.11

Suggestion: please obfuscate filenames when importing icons.

[droidmonkey]

There is no filename in the database after importing an icon.

We use Qt to render the icon, so if it does not render correctly it is likely an SVG issue or setting that is causing it. We request a rendering of 128x128 resolution in PNG format with antialiasing.

[t214c]

Maybe not in the UI. I discovered it while trying to change an icon in KeePassDX. I originally added that custom icon in XC. The name is saved.

[droidmonkey]

There is no name for custom icons. Did you add it as an attachment? Either way there is zero reason to obfuscate anything, everything in a keepass database is encrypted.

[t214c]

• Add an icon named test.png to group A in XC.
• Sync
• Open DB in DX, long tap group B > menu > Edit > Tap icon > Custom.

You will see the name as test under the icon added from the desktop.

everything in a keepass database is encrypted.

I have to agree with you there. It is just both of a redundancy-in-security itch and neatness! :)

[droidmonkey]

Oh interesting, I was wrong we do store the base file name. Either way that information is encrypted.

[t214c]

That covers the security aspect. As for the neatness, I'll have to live with my ugly temporary filenames etched into the database indefinitely! :)

[droidmonkey]

You can edit them in DX

[t214c]

Oh! I didn't realize that. Thanks!