AllChecksOK does not mean that all checks are ok

matrix-org / matrix-federation-tester

Tester for matrix federation written in golang.

79 stars 17 forks source link

Closed richvdh closed 5 years ago

richvdh commented 5 years ago

Apparently some checks aren't included in AllChecksOK, which is super misleading

anoadragon453 commented 5 years ago

ValidCertificate wasn't included as it's not mandatory at the moment (it's technically fine if you have self-signed certs).

But maybe it should be because deadline is coming up for them?

richvdh commented 5 years ago

Oh, is that the only thing that's not included?

I think it should be, anyway.

anoadragon453 commented 5 years ago

Can do.

richvdh commented 5 years ago

related: #11