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

Remove deprecated bank_rule.yar #415

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

Conversation

mnalis
Copy link

@mnalis mnalis commented Aug 8, 2022

it has been removed in upstream Yara rules, and so we currently always throw warning:

WARNING: Failed connection to https://raw.githubusercontent.com/Yara-Rules/rules/master - SKIPPED yararulesproject bank_rule.yar update

PR is made against dev branch.

Also, it would be good to finally publish dev branch to master and make a release, as there are few other bugs pending which users keep reporting (like #400, #414, etc)

@perplexityjeff
Copy link
Contributor

I applied the patch locally, thanks for noticing. I agree with you that it has been a long time until a release and some minor patches are already in dev waiting for release, I hope @extremeshok finds time to maintain this project again.

dimejo added a commit to dimejo/clamav-unofficial-sigs that referenced this pull request Feb 19, 2023
Original PR: extremeshok#415


it has been removed in upstream Yara rules, and so we currently always throw warning:

    WARNING: Failed connection to https://raw.githubusercontent.com/Yara-Rules/rules/master - SKIPPED yararulesproject bank_rule.yar update

PR is made against dev branch.

Also, it would be good to finally publish dev branch to master and make a release, as there are few other bugs pending which users keep reporting (like extremeshok#400, extremeshok#414, etc)
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.

2 participants