Fix pg_hba_file_rules for authentication method cert

mhagander · mhagander · commit aa58f5c53142 · 2022-01-26T09:59:23.000+01:00
For authentication method cert, clientcert=verify-full is implied. But
the pg_hba_file_rules entry would incorrectly show clientcert=verify-ca.

Per bug #17354

Reported-By: Feike Steenbergen
Reviewed-By: Jonathan Katz
Backpatch-through: 12
diff --git a/src/backend/libpq/hba.c b/src/backend/libpq/hba.c
@@ -1632,7 +1632,11 @@ parse_hba_line(TokenizedLine *tok_line, int elevel)
 	 */
 	if (parsedline->auth_method == uaCert)
 	{
-		parsedline->clientcert = clientCertCA;
+		/*
+		 * For auth method cert, client certificate validation is mandatory, and it implies
+		 * the level of verify-full.
+		 */
+		parsedline->clientcert = clientCertFull;
 	}
 
 	return parsedline;

Original file line number	Diff line number	Diff line change
`@@ -1632,7 +1632,11 @@ parse_hba_line(TokenizedLine *tok_line, int elevel)`
`1632`	`1632`	`*/`
`1633`	`1633`	`if (parsedline->auth_method == uaCert)`
`1634`	`1634`	`{`
`1635`		`- parsedline->clientcert = clientCertCA;`
	`1635`	`+ /*`
	`1636`	`+ * For auth method cert, client certificate validation is mandatory, and it implies`
	`1637`	`+ * the level of verify-full.`
	`1638`	`+ */`
	`1639`	`+ parsedline->clientcert = clientCertFull;`
`1636`	`1640`	`}`
`1637`	`1641`
`1638`	`1642`	`return parsedline;`