Andrew Stitcher

Results 32 comments of Andrew Stitcher

In response to some feedback I've made the full frame logging the default (and only) behaviour when frame tracing is turned on (PN_LOG=frame). The extra control wasn't very useful, at...

My intention is to split out the fixed string work from this branch as it isn't really the focus of this WIP, but an independently useful change which gos to...

You don't explain what problem you are solving with these changes. Please give more details about why you need to - 1. change the case of the includes - I...

> > 1. **File name casing**. The windows API is not really very consistent when is comes to the casing of file names (can differ per VS version, or platform),...

Having said all that on the whole if the Windows CI build works and the Linux CI build is unaffected I'm ok with merging this change. I'd just prefer to...

@josusky Cyrus SASL may well be multiplatfrom, but there is no readily available of the library for Windows as far as I can tell. It is not available on vcpkg...

> vcpkg request for cyrus sasl, [microsoft/vcpkg#5894](https://github.com/microsoft/vcpkg/issues/5894), currently abandoned Unfortunately this is only an issue, rather than an actual PR with the changes necessary to actually build cyrus under vcpkg...

Having checked the AMQP spec there is no default value for TTL. I think that proton-c just reuses 0 to mean no ttl header because sending a ttl of 0...