Profiler log output should use strict JSON representation for BSON types

[jmikola]

From comment in #177:

I suppose the most preferable option would be to serialize the driver classes in the "strict JSON" format defined in: http://docs.mongodb.org/manual/reference/mongodb-extended-json/#data_binary (the mongoexport commands use that notation).

[stale[bot]]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[jmikola]

@malarzm: It's been a while since I worked with the bundle, but are CommandDataCollector.php and PSRCommandLogger.php the only classes relevant to data collection and logging? The original classes mentioned in #177 no longer exist.

It looks like the current classes are using json_encode(), which addresses the original concern in #177 since binary data is displayed as base64-encoded. That said, the driver's JsonSerializable implementations on BSON types yields a legacy extended JSON format (vs. the formats outlined in the Extended JSON spec). If you think it's worth fixing that, I can submit a quick patch that round-trips the command documents through fromPHP() and toRelaxedExtendedJSON().

If not, feel free to close this as-is.

[malarzm]

@jmikola looking at Conversion table the relaxed format looks nice and if it's not a big hassle I'd say it's nice to have :)

[alcaeus]

Let's try to get this done for 4.0 :)

[IonBazan]

@alcaeus isn't it fixed in #652?