8000 Fix advertised dispsize for libpq's sslmode connection parameter. · Deepakkothandan/postgres@962c148 · GitHub
[go: up one dir, main page]

Skip to content

Commit 962c148

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 6e6c2c2 commit 962c148

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
@@ -181,7 +181,7 @@ static const PQconninfoOption PQconninfoOptions[] = {
181181
* to exclude them since none of them are mandatory.
182182
*/
183183
{"sslmode", "PGSSLMODE", DefaultSSLMode, NULL,
184-
"SSL-Mode", "", 8}, /* sizeof("disable") == 8 */
184+
"SSL-Mode", "", 12}, /* sizeof("verify-full") == 12 */
185185

186186
{"sslcert", "PGSSLCERT", NULL, NULL,
187187
"SSL-Client-Cert", "", 64},

0 commit comments

Comments
 (0)
0