astitcher commented on PR #438: URL: https://github.com/apache/qpid-proton/pull/438#issuecomment-2668837181
@pjfawcett Thanks for jogging my attention back here! I've been thinking about this a little bit and I think I;'ve concluded that the surrogateescape handling should actually be the default for conversions from binary to python strings in the very few places we do it in both directions, mostly for backwards compatibility to the previous swig binding. So I'll take this PR and modify it slightly in that direction. Thanks for your work on this. If you want to create a separate PR to add a btag property to deliveries I'd accept that - but currently I'm thinking that I'll change/fix this API when we move to a 1.0 version to return a bytes object (this should be in the next 6-9 months). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org
