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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

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.

@@ -2,7 +2,7 @@
<package xmlns="http://schemas.microsoft.com/packaging/2015/06/nuspec.xsd">
<metadata>
<id>cyberchef.vm</id>
<version>10.18.3</version>
<version>10.18.3.20240523</version>
Copy link
Member

@Ana06 Ana06 May 28, 2024

Choose a reason for hiding this comment

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

It needs update because #1052 has been merged, please rebase 😉

@Ana06
Copy link
Member

Ana06 commented Jun 18, 2024

@emtuls 👀

…-category

Fix the calls to VM-Uninstall-With-Uninstaller
@@ -19,13 +19,13 @@ try {
Install-ChocolateyZipPackage @packageArgs
VM-Assert-Path $toolDir

$iconLocation = VM-Create-Ico (Join-Path $toolDir "images\cyberchef-128x128.png") # Create .ico for cyberchef icon
Copy link
Member

Choose a reason for hiding this comment

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

@emtuls have you forgotten to push the updated changes? 🤔 This code is still creating the .ico instead of using https://raw.githubusercontent.com/gchq/CyberChef/18159ce806e63286019a5a6828422a992fbb253b/src/web/static/images/favicon.ico that seems to looks better and does not require the extra function

@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.

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?
3 participants