[DRIVERS-427] endSessions command no longer takes `ids` parameter Created: 20/Nov/17 Updated: 28/Oct/23 Resolved: 05/Jun/18 |
|
| Status: | Closed |
| Project: | Drivers |
| Component/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Matt Broadstone | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | newdriver | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||
| Description |
|
This is something I missed that changed in the spec recently. The problem here is that the advice in the spec is to explicitly ignore all responses to the endSessions command, so I never saw that my code was broken until I was looking at verbose server logs. We should probably make sure all drivers caught this so we're not leaking sessions. |
| Comments |
| Comment by Rathi Gnanasekaran [ 05/Jun/18 ] | |
|
Closed via automated process. | |
| Comment by A. Jesse Jiryu Davis [ 20/Nov/17 ] | |
|
Right, the session ids are an array, the value of the "endSessions" field:
The C Driver tests endSessions thus:
Checking the server reply in this test will catch a command syntax error like this. |