Content-Length: 252129 | pFad | http://github.com/postgres/postgres/commit/962c14894768f95fabc82d4fb7db38c76d88fb97

96 Fix advertised dispsize for libpq's sslmode connection parameter. · postgres/postgres@962c148 · GitHub
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)








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgres/postgres/commit/962c14894768f95fabc82d4fb7db38c76d88fb97

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy