forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit9bb5eec
committed
Avoid ECPG test failures in some GSS-capable environments.
Buildfarm member hamerkop has been reporting that two cases inconnect/test5.pgc show different error messages than the test expects,because since commitffa2e46 libpq's connection failure messagesare exposing the fact that a GSS-encrypted connection was attemptedand failed. That's pretty interesting information in itself, andI certainly don't wish to shoot the messenger, but we need to dosomething to stabilize the ECPG results.For the second of these two failure cases, we can add thegssencmode=disable option to prevent the discrepancy. However,that solution is problematic for the first failure, because the onlyunique thing about that case is that it's testing a completely-omittedconnection target; there's noplace to add the option without defeatingthe point of the test case. After some thrashing around withalternative fixes that turned out to have undesirable side-effects,the most workable answer is just to give up and remove that test case.Perhaps we can revert this later, if we figure out why the GSS codeis misbehaving in hamerkop's environment.Thanks to Michael Paquier for exploration of alternatives.Discussion:https://postgr.es/m/YLRZH6CWs9N6Pusy@paquier.xyz1 parentb29fa95 commit9bb5eec
File tree
3 files changed
+7
-21
lines changed- src/interfaces/ecpg/test
- connect
- expected
3 files changed
+7
-21
lines changedLines changed: 3 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
40 | 40 |
| |
41 | 41 |
| |
42 | 42 |
| |
43 |
| - | |
44 |
| - | |
| 43 | + | |
| 44 | + | |
45 | 45 |
| |
46 | 46 |
| |
47 | 47 |
| |
| |||
61 | 61 |
| |
62 | 62 |
| |
63 | 63 |
| |
64 |
| - | |
| 64 | + | |
65 | 65 |
| |
66 | 66 |
| |
67 | 67 |
| |
|
Lines changed: 3 additions & 7 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
86 | 86 |
| |
87 | 87 |
| |
88 | 88 |
| |
89 |
| - | |
90 |
| - | |
91 |
| - | |
92 |
| - | |
93 |
| - | |
94 |
| - | |
| 89 | + | |
| 90 | + | |
95 | 91 |
| |
96 | 92 |
| |
97 | 93 |
| |
| |||
135 | 131 |
| |
136 | 132 |
| |
137 | 133 |
| |
138 |
| - | |
| 134 | + | |
139 | 135 |
| |
140 | 136 |
| |
141 | 137 |
| |
|
Lines changed: 1 addition & 11 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
34 | 34 |
| |
35 | 35 |
| |
36 | 36 |
| |
37 |
| - | |
38 |
| - | |
39 |
| - | |
40 |
| - | |
41 |
| - | |
42 |
| - | |
43 |
| - | |
44 |
| - | |
45 |
| - | |
46 |
| - | |
47 | 37 |
| |
48 | 38 |
| |
49 | 39 |
| |
| |||
70 | 60 |
| |
71 | 61 |
| |
72 | 62 |
| |
73 |
| - | |
| 63 | + | |
74 | 64 |
| |
75 | 65 |
| |
76 | 66 |
| |
|
0 commit comments
Comments
(0)