Skip to content

Define interval to retry to update CVE databases#792

Merged
bah07 merged 1 commit into3.3from
fix-issue-768
Jun 15, 2018
Merged

Define interval to retry to update CVE databases#792
bah07 merged 1 commit into3.3from
fix-issue-768

Conversation

@chemamartinez
Copy link
Contributor

Related to the issue: #768

Now when the vulnerability-detector module can't update the CVE databases, it makes another attempt five minutes later. If it also fails, it waits for the defined interval in the configuration for each distribution.

For example, if we define an hour interval for updating the RedHat 7 database:

2018/06/13 09:50:55 wazuh-modulesd:vulnerability-detector: INFO: (5461): Starting Red Hat Enterprise Linux 7 DB update...
2018/06/13 09:50:55 wazuh-modulesd:vulnerability-detector: DEBUG: (5450): Downloading RHEL7 database...
2018/06/13 09:50:55 wazuh-modulesd:vulnerability-detector: ERROR: (5400): RHEL7 database could not be fetched.
2018/06/13 09:50:55 wazuh-modulesd:vulnerability-detector: DEBUG: Failed when updating RedHat databases. Retrying in 300 seconds...
2018/06/13 09:50:55 wazuh-modulesd:vulnerability-detector: ERROR: (5426): CVE database could not be updated.

...

2018/06/13 09:55:55 wazuh-modulesd:vulnerability-detector: INFO: (5461): Starting Red Hat Enterprise Linux 7 DB update...
2018/06/13 09:55:55 wazuh-modulesd:vulnerability-detector: DEBUG: (5450): Downloading RHEL7 database...
2018/06/13 09:55:55 wazuh-modulesd:vulnerability-detector: ERROR: (5400): RHEL7 database could not be fetched.
2018/06/13 09:55:55 wazuh-modulesd:vulnerability-detector: DEBUG: Failed when updating RedHat databases. Retrying in 3600 seconds...
2018/06/13 09:55:55 wazuh-modulesd:vulnerability-detector: ERROR: (5426): CVE database could not be updated.

@jesusgn90
Copy link
Contributor

Great @chemamartinez !

@bah07 bah07 merged commit e03421f into 3.3 Jun 15, 2018
@bah07 bah07 deleted the fix-issue-768 branch June 15, 2018 11:19
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