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

Scan does not remove a deactivated SSL server #21

Open
Tuxdiver opened this Issue Dec 6, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@Tuxdiver
Copy link

Tuxdiver commented Dec 6, 2018

I deactivated some ssl webservers, but a new scan does not remove them from the module's database

Expected Behavior

Scan should remove old entries

Current Behavior

Scan does NOT remove old entries

Possible Solution

Make it so!

Steps to Reproduce (for bugs)

  • Scan for SSL hosts
  • deactivate Host (ifdown IP)
  • Scan again
  • Host still in x509 module list

Your Environment

  • Module version (System - About): git master a384a29
  • Icinga Web 2 version and modules (System - About): 2.6.1
  • Icinga 2 version (icinga2 --version): r2.10.2-1
  • Operating System and version: Ubuntu 16.04
@Napsty

This comment has been minimized.

Copy link

Napsty commented Jan 7, 2019

Or (as a workaround) add a checkbox or drop-down menu to manually delete a certificate from the module database.

@Al2Klimov Al2Klimov self-assigned this Jan 15, 2019

@Al2Klimov Al2Klimov referenced a pull request that will close this issue Jan 16, 2019

Open

Job#startNextTarget(): correct SQL UPDATE filter #31

@Al2Klimov

This comment has been minimized.

Copy link
Contributor

Al2Klimov commented Jan 16, 2019

Hello @Tuxdiver and thank you for reporting!

Please could you test #31?

Best,
AK

@Tuxdiver

This comment has been minimized.

Copy link

Tuxdiver commented Jan 16, 2019

Certificates are removed in "Certificate Usage" but not in "Certificate Overview".
I even find expired "lets encrypt" certs there.

Is that a bug or a feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment