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 malpedia galaxy #877

Merged
merged 1 commit into from
Oct 31, 2023
Merged

update malpedia galaxy #877

merged 1 commit into from
Oct 31, 2023

Conversation

HiS3
Copy link
Contributor

@HiS3 HiS3 commented Oct 19, 2023

No description provided.

@@ -13,7 +13,7 @@
"name": "Malpedia",
"source": "Malpedia",
"type": "malpedia",
"uuid": "1d1c9af9-37fa-4deb-a928-f9b0abc7354a",
Copy link
Member

Choose a reason for hiding this comment

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

Not sure why you changed the UUID, it should remain fixed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hm Curious .. this came automaticly from the fetch_malpedia.sh script

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if i look right, the uuid come from Malpedia itself
https://malpedia.caad.fkie.fraunhofer.de/api/get/misp

Copy link
Member

Choose a reason for hiding this comment

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

@danielplohmann maybe we should keep the UUID of the cluster from the malpedia api. What do you think?

Copy link
Contributor

@danielplohmann danielplohmann Oct 26, 2023

Choose a reason for hiding this comment

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

I just had a look at our code base, the generated galaxy accessible from our API endpoint should always come with UUID 5fc98d08-90a4-498a-ad2e-0edf50ef374e (git blame says, at least since 2018-10-10) for consistency reasons, only the content and version field changes in accordance with our internal commit count.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, so that means @danielplohmann that 5fc98d08-90a4-498a-ad2e-0edf50ef374e are the correct UUID ?

Copy link
Member

Choose a reason for hiding this comment

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

@danielplohmann It should be the current one in the misp repository. If not it will be see as a new malpedia galaxy. Could you use the current one? Thank you

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, no problem, we will change it on our server to match the UUID that you already have!
Should be deployed in the next days with an upcoming update we are preparing anyway. 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

for completeness, we finally deployed a big update yesterday which should also fix the UUID issue that was raised here.

@adulau adulau merged commit c800ad0 into MISP:main Oct 31, 2023
4 checks passed
@adulau
Copy link
Member

adulau commented Oct 31, 2023

Thanks for the contribution. I merged it with the updated UUID.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants