TryGetDocumentId returns string on failure

[kierenj]

Being picky, but a Try* operation is expected to return a boolean indicating success or failure, with an output parameter to deal with the result. Returning a hardcoded string upon failure, which must be compared manually by the caller to a hardcoded string, seems a bad path to go down?

[g0t4]

I'd suggest using TryGetValue("_id", out id); provided by the mongodb driver for that behavior.

[cd21h]

Agree with @g0t4