- Notifications
You must be signed in to change notification settings - Fork28
Commiteb882a1
committed
GSSAPI: Improve documentation and tests
The GSSAPI encryption patch neglected to update the protocoldocumentation to describe how to set up a GSSAPI encrypted connectionfrom a client to the server, so fix that by adding the appropriatedocumentation to protocol.sgml.The tests added for encryption support were overly long and couldn't berun in parallel due to race conditions; this was largely because eachtest was setting up its own KDC to perform the tests. Instead, mergethe authentication tests and the encryption tests into the originaltest, where we only create one KDC to run the tests with. Also, havethe tests check what the server's opinion is of the connection and if itwas GSS authenticated or encrypted using the pg_stat_gssapi view.In passing, fix the libpq label for GSSENC-Mode to be consistent withthe "PGGSSENCMODE" environment variable.Missing protocol documentation pointed out by Michael Paquier.Issues with the tests pointed out by Tom Lane and Peter Eisentraut.Refactored tests and added documentation by me.Reviewed by Robbie Harwood (protocol documentation) and Michael Paquier(rework of the tests).1 parentb8b94ea commiteb882a1
File tree
4 files changed
+216
-211
lines changed- doc/src/sgml
- src
- interfaces/libpq
- test/kerberos/t
4 files changed
+216
-211
lines changedLines changed: 106 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1479 | 1479 |
| |
1480 | 1480 |
| |
1481 | 1481 |
| |
| 1482 | + | |
| 1483 | + | |
| 1484 | + | |
| 1485 | + | |
| 1486 | + | |
| 1487 | + | |
| 1488 | + | |
| 1489 | + | |
| 1490 | + | |
| 1491 | + | |
| 1492 | + | |
| 1493 | + | |
| 1494 | + | |
| 1495 | + | |
| 1496 | + | |
| 1497 | + | |
| 1498 | + | |
| 1499 | + | |
| 1500 | + | |
| 1501 | + | |
| 1502 | + | |
| 1503 | + | |
| 1504 | + | |
| 1505 | + | |
| 1506 | + | |
| 1507 | + | |
| 1508 | + | |
| 1509 | + | |
| 1510 | + | |
| 1511 | + | |
| 1512 | + | |
| 1513 | + | |
| 1514 | + | |
| 1515 | + | |
| 1516 | + | |
| 1517 | + | |
| 1518 | + | |
| 1519 | + | |
| 1520 | + | |
| 1521 | + | |
| 1522 | + | |
| 1523 | + | |
| 1524 | + | |
| 1525 | + | |
| 1526 | + | |
| 1527 | + | |
| 1528 | + | |
| 1529 | + | |
| 1530 | + | |
| 1531 | + | |
| 1532 | + | |
| 1533 | + | |
| 1534 | + | |
| 1535 | + | |
| 1536 | + | |
| 1537 | + | |
| 1538 | + | |
| 1539 | + | |
| 1540 | + | |
| 1541 | + | |
| 1542 | + | |
| 1543 | + | |
| 1544 | + | |
| 1545 | + | |
| 1546 | + | |
| 1547 | + | |
| 1548 | + | |
| 1549 | + | |
| 1550 | + | |
1482 | 1551 |
| |
1483 | 1552 |
| |
1484 | 1553 |
| |
| |||
5714 | 5783 |
| |
5715 | 5784 |
| |
5716 | 5785 |
| |
| 5786 | + | |
| 5787 | + | |
| 5788 | + | |
| 5789 | + | |
| 5790 | + | |
| 5791 | + | |
| 5792 | + | |
| 5793 | + | |
| 5794 | + | |
| 5795 | + | |
| 5796 | + | |
| 5797 | + | |
| 5798 | + | |
| 5799 | + | |
| 5800 | + | |
| 5801 | + | |
| 5802 | + | |
| 5803 | + | |
| 5804 | + | |
| 5805 | + | |
| 5806 | + | |
| 5807 | + | |
| 5808 | + | |
| 5809 | + | |
| 5810 | + | |
| 5811 | + | |
| 5812 | + | |
| 5813 | + | |
| 5814 | + | |
| 5815 | + | |
| 5816 | + | |
| 5817 | + | |
| 5818 | + | |
| 5819 | + | |
| 5820 | + | |
| 5821 | + | |
| 5822 | + | |
5717 | 5823 |
| |
5718 | 5824 |
| |
5719 | 5825 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
313 | 313 |
| |
314 | 314 |
| |
315 | 315 |
| |
316 |
| - | |
| 316 | + | |
317 | 317 |
| |
318 | 318 |
| |
319 | 319 |
| |
|
Lines changed: 109 additions & 13 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
1 | 14 |
| |
2 | 15 |
| |
3 | 16 |
| |
| |||
6 | 19 |
| |
7 | 20 |
| |
8 | 21 |
| |
9 |
| - | |
| 22 | + | |
10 | 23 |
| |
11 | 24 |
| |
12 | 25 |
| |
| |||
50 | 63 |
| |
51 | 64 |
| |
52 | 65 |
| |
53 |
| - | |
| 66 | + | |
54 | 67 |
| |
55 | 68 |
| |
56 | 69 |
| |
| |||
155 | 168 |
| |
156 | 169 |
| |
157 | 170 |
| |
158 |
| - | |
| 171 | + | |
| 172 | + | |
159 | 173 |
| |
160 | 174 |
| |
161 |
| - | |
| 175 | + | |
162 | 176 |
| |
163 |
| - | |
| 177 | + | |
164 | 178 |
| |
165 |
| - | |
166 |
| - | |
167 |
| - | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
168 | 184 |
| |
169 |
| - | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
170 | 195 |
| |
171 | 196 |
| |
172 | 197 |
| |
| |||
175 | 200 |
| |
176 | 201 |
| |
177 | 202 |
| |
178 |
| - | |
| 203 | + | |
179 | 204 |
| |
180 | 205 |
| |
181 | 206 |
| |
182 |
| - | |
| 207 | + | |
183 | 208 |
| |
184 | 209 |
| |
185 | 210 |
| |
186 | 211 |
| |
187 |
| - | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
188 | 278 |
| |
189 | 279 |
| |
190 | 280 |
| |
191 | 281 |
| |
192 | 282 |
| |
193 | 283 |
| |
194 | 284 |
| |
195 |
| - | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + |
0 commit comments
Comments
(0)