Skip to content

Tweak suffixes#475

Merged
pyth0n1c merged 2 commits intomainfrom
appinspect_fix
Mar 5, 2026
Merged

Tweak suffixes#475
pyth0n1c merged 2 commits intomainfrom
appinspect_fix

Conversation

@ljstella
Copy link
Contributor

@ljstella ljstella commented Mar 5, 2026

Looks like Splunkbase changed some things, so metadata validation was hitting a ValueError and bailing out. See https://github.com/splunk/security_content/actions/runs/22721988462/job/65886387827?pr=3937 for example.

@ljstella
Copy link
Contributor Author

ljstella commented Mar 5, 2026

Successful now with these changes:

Screenshot 2026-03-05 at 11 15 14 AM

Copy link
Contributor

@cmcginley-splunk cmcginley-splunk left a comment

Choose a reason for hiding this comment

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

Good catch! Confirmed on my end that the suffixes used by Splunkbase on download have changed

Copy link
Contributor

@pyth0n1c pyth0n1c left a comment

Choose a reason for hiding this comment

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

I have confirmed with the Splunkbase Team that this is an intentional change on their end/ Great catch, thanks Lou!
Approved.

@pyth0n1c pyth0n1c merged commit 09442c2 into main Mar 5, 2026
16 checks passed
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