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.

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

Update CyberChef to use CyberChef icon #1054

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Update CyberChef to use CyberChef icon #1054

merged 1 commit into from
Oct 1, 2024

Conversation

emtuls
Copy link
Member

@emtuls emtuls commented May 23, 2024

This converts the CyberChef PNG file that is provided with the download and converts it to a .ico so that Chocolatey can use it for an icon for the shortcut.

Closes #966

@emtuls emtuls added 💎 enhancement It is working, but it could be better 🌀 FLARE-VM A package or feature to be used by FLARE-VM labels May 23, 2024
@emtuls emtuls self-assigned this May 23, 2024
@emtuls emtuls requested review from Ana06 and mr-tz May 23, 2024 19:58
@Ana06 Ana06 mentioned this pull request May 24, 2024
packages/cyberchef.vm/tools/chocolateyinstall.ps1 Outdated Show resolved Hide resolved
packages/cyberchef.vm/tools/chocolateyinstall.ps1 Outdated Show resolved Hide resolved
Copy link
Member

@Ana06 Ana06 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes @emtuls! After some more testing, I am not sure that converting the image to icon is the best approach. The current code make the shortcut looks like:
image
But we could download and use directly the icon in https://raw.githubusercontent.com/gchq/CyberChef/18159ce806e63286019a5a6828422a992fbb253b/src/web/static/images/favicon.ico which I think looks a bit better and removes the need for the extra code:
image

It is also confusing and inconsistent with other apps (like capa or fakenet) that the icon in the taskbar is a different one. This is related to #1021 and #822, but I think we should address then in the same PR so that the icon change takes place everywhere.

@Ana06
Copy link
Member

Ana06 commented Jun 18, 2024

@emtuls 👀

@emtuls emtuls force-pushed the cyberchef-update branch 2 times, most recently from 221b105 to 1b8dcd8 Compare June 20, 2024 16:40
@emtuls emtuls requested a review from Ana06 June 20, 2024 16:45
@emtuls
Copy link
Member Author

emtuls commented Jul 8, 2024

@mandiant/flare-vm Any preference or thoughts on which icon to use for Cyberchef?
One icon uses the robot-chef mix. It looks good on taskbar and on the desktop but has a slight blurry look when viewed in the Tools folder in small/detail view.
cyberchef
image

The other icon is just a Chef hat (also provided by Cyberchef as a .ico, so not a random chef hat), which looks a little better in small/detail view, but I personally think it looks a bit blurry when viewed on Desktop and the Taskbar.
image
image

Another thing to note is that the Robot-Chef mix icon requires some additional code to convert an image file to a .ico file.

@Ana06
Copy link
Member

Ana06 commented Jul 25, 2024

@d35ha can you test this locally and share your opinion on this? 😄

@Ana06
Copy link
Member

Ana06 commented Aug 26, 2024

@stevemk14ebr @mr-tz @binjo @d35ha any preference on the icon? The icons look a bit different (see screenshots in the previous comments). @emtuls prefers to generate Robot-Chef icon (requires some additional code to convert an image file to a .ico file), while I prefer the Chef hat.

@stevemk14ebr
Copy link
Contributor

I don't mind either, I vote for whichever is easiest to maintain

@mr-tz
Copy link
Contributor

mr-tz commented Aug 29, 2024

I don't mind either, I vote for whichever is easiest to maintain

I second this. FWIW taskbars may be configured in other colors (mine is black). I agree that gray on gray is a bit hard to see.

@Ana06
Copy link
Member

Ana06 commented Aug 29, 2024

I think the Chef hat is easier to maintain because it requires less code. But as it seems nobody else has a strong opinion, I'll let @emtuls decide. @emtuls can you please rebase the final version so that we can merge this change? 😄

@emtuls
Copy link
Member Author

emtuls commented Sep 13, 2024

After googling CyberChef and seeing most websites/videos use the Robot-Chef icon, I'm inclined to keep that style for now. :)

@emtuls emtuls force-pushed the cyberchef-update branch 4 times, most recently from 9dfdcd4 to 81433e7 Compare September 13, 2024 22:57
Copy link
Member

@Ana06 Ana06 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

I find it confusing to have the CyberChef icon in the taskbar and that when clicking it, it opens Chrome as a different taskbar icon:

image

But I think this is already part of #1100 and we can continue to discuss how to address it there.

@Ana06 Ana06 merged commit 622c271 into main Oct 1, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌀 FLARE-VM A package or feature to be used by FLARE-VM 💎 enhancement It is working, but it could be better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use CyberChef icon instead of Chrome icon for CyberChef?
4 participants