32feet
32feet copied to clipboard
added ObexListenerContext(Stream rs) for Android file transfer
Added ObexListenerContext(Stream rs) for file transfer between Windows and Android due to System.Net.Sockets not being compatible with Android sockets. added ContentMd5 ObexHeader to facilitate sending Md5 checksum to destination with file.
No problem Peter, I only get to look at it occasionally myself. I will test to make sure I can achieve what we require with your changes. Are they changes included on nuget.org yet or should I use the code from main on Git? Thanks for all your help. Regards, David
From: Peter Foot @.> Sent: Saturday, November 25, 2023 4:50 AM To: inthehand/32feet @.> Cc: David Rodgers @.>; Author @.> Subject: Re: [inthehand/32feet] added ObexListenerContext(Stream rs) for Android file transfer (PR #356)
@peterfoot commented on this pull request.
In InTheHand.Net.Obex/ObexWebRequest.cshttps://github.com/inthehand/32feet/pull/356#discussion_r1404591547:
@@ -1038,8 +1050,16 @@ public override WebResponse GetResponse()
ObexStatusCode status;
MemoryStream ms = new MemoryStream();
WebHeaderCollection responseHeaders = new WebHeaderCollection();
-
// Added David Rodgers to facilitate sending an Md5 checksum in the header
-
// could be expanded to loop through Headers and add any headers that are set by the user
-
if (Headers["ContentMd5"] != null)
Sorry I've been distracted and forgot to follow up on this. Now that the listener can work from either a stream or a socket, I don't think the public constructor is needed because this can be created through the normal listener flow.
— Reply to this email directly, view it on GitHubhttps://github.com/inthehand/32feet/pull/356#discussion_r1404591547, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AY6BX3NQJJRXO2Q4FDSUFFDYGDT57AVCNFSM6AAAAAA6QEFR5CVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBYGMZTKNZYGY. You are receiving this because you authored the thread.Message ID: @.@.>>