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

Skip to content

Commit 0bdeb1d

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 f2063b3 commit 0bdeb1d

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
@@ -220,7 +220,7 @@ static const PQconninfoOption PQconninfoOptions[] = {
220220
* to exclude them since none of them are mandatory.
221221
*/
222222
{"sslmode", "PGSSLMODE", DefaultSSLMode, NULL,
223-
"SSL-Mode", "", 8}, /* sizeof("disable") == 8 */
223+
"SSL-Mode", "", 12}, /* sizeof("verify-full") == 12 */
224224

225225
{"sslcompression", "PGSSLCOMPRESSION", "1", NULL,
226226
"SSL-Compression", "", 1},

0 commit comments

Comments
 (0)