betfair / cougar

Cougar is a framework for making building network exposed service interfaces easy.
http://betfair.github.io/cougar
Apache License 2.0
27 stars 18 forks source link

Cougar logger inserts comma's while rendering collections #22

Closed eswdd closed 9 years ago

eswdd commented 10 years ago

Cougar uses "," (comma) as separator while rendering collections. This leads to request logs of variable field length which will break any downstream log parser tools, such as ones for importing into a database.

We should use the pipe character as the default.

eswdd commented 9 years ago

Change had already been made to make this configurable during the open sourcing. Now am changing the default value of the controlling property "cougar.eventLogRecord.collectionSeperator"