That just leaves the last TODO, which is guiding users towards a resolution. First attempt at that is attached as v7: - v7-0001 is the same as v6-0003 with an updated commit message. - v7-0002 is a squash! commit that prints a quick explanation and a link to our documentation for all grease-related libpq errors.
I'd still prefer that this link go to a wiki page, but in case I can't get agreement on that, or there's some technical reason why protecting the page wouldn't work out the way I'd hoped, or it just takes a long time, this seems better than nothing. Switching out the URL in a later commit shouldn't be an issue.
WDYT?
Looks like a good improvement. As said before I'm fine with either location for the URL.
I'm wondering if we should be a bit more liberal in showing this error explanation, since I expect most servers to throw an error rather than send an incorrect negotiation. e.g. We could report it on hard connection closures. Or if there is "3.9999" or "version" or "_pq_" in the error message. (the message should then be "this could indicate a bug in..." because we're not fully certain)