Skip to content
This repository has been archived by the owner on Mar 16, 2023. It is now read-only.

Adding an expire time when updating the Cofense intelligence feed #298

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mcmahonr
Copy link

@mcmahonr mcmahonr commented Oct 5, 2018

This will make sure that old data isn't accidentally imported if the API has an issue in the future

Recently our ThreatHQ API had an issue where it was updating with threats that were very old. We're adding a check to make sure threats being imported during an update are no older than 90 days.
Adding a default expire time of 90 days
@jtschichold
Copy link
Member

Hi @mcmahonr,
thanks for the PR, please:

  • target the develop branch in the PR, not master
  • add your name to the AUTHORS file
  • in the PR comment explicitly state that you accept the license

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

Successfully merging this pull request may close these issues.

2 participants