Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 37cb060

Browse files
committed
Fix advertised dispsize for libpq's sslmode connection parameter.
"8" was correct back when "disable" was the longest allowed value, but since "verify-full" was added, it should be "12". Given the lack of complaints, I wouldn't be surprised if nobody is actually using these values ... but still, if they're in the API, they should be right. Noticed while pursuing a different problem. It's been wrong for quite a long time, so back-patch to all supported branches.
1 parent b5de169 commit 37cb060

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/interfaces/libpq/fe-connect.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ static const internalPQconninfoOption PQconninfoOptions[] = {
251251
* to exclude them since none of them are mandatory.
252252
*/
253253
{"sslmode", "PGSSLMODE", DefaultSSLMode, NULL,
254-
"SSL-Mode", "", 8, /* sizeof("disable") == 8 */
254+
"SSL-Mode", "", 12, /* sizeof("verify-full") == 12 */
255255
offsetof(struct pg_conn, sslmode)},
256256

257257
{"sslcompression", "PGSSLCOMPRESSION", "1", NULL,

0 commit comments

Comments
 (0)