search

matrix-org / matrix-federation-tester

Tester for matrix federation written in golang.
78 stars 17 forks source link

Change error to result in wellknownreport. Less scary if no well-known #70

Closed anoadragon453 closed 5 years ago

anoadragon453 commented 5 years ago

Users no longer see:

  "WellKnownResult": {
    "m.server": "",
    "error": "No .well-known found"
  },

but instead:

  "WellKnownResult": {
    "m.server": "",
    "result": "No .well-known found"
  },

Closes #50

aaronraimist commented 5 years ago

Did you see the suggestion someone made (sorry I forgot who), instead of saying individually that well-known or SRV wasn’t found, say that the home server was discovered either directly on 8448, via a SRV record, or via well-known (or SRV and well-known)?

That way it is less scary. People who think they have a well-known in place but see that it was only discovered by SRV would know that something was wrong with their well known but people who don’t need a well-known (or don’t need either SRV or well-known) wouldn’t need to worry that they missed setting up something important.

anoadragon453 commented 5 years ago

I think both changes will make things less scary, but that suggestion sounds like a good long-term fix. Could you turn it into a separate issue so we don't forget about it?