FreeRDP icon indicating copy to clipboard operation
FreeRDP copied to clipboard

implement external io handling (networking, crypto, ...)

Open sss123next opened this issue 5 years ago • 3 comments

it's required to do whole io handling outside libfreerdp for both servers and clients implementations.

with current api this is seems to be impossible, so we currently working on implementation this kind of api for freerdp, goal is to maintain complete compatibility with existing codebase.

currently i am doing research of freerdp internals. for now i see it as:

  1. add BOOL? variable to switch io handling method
  2. add write function somewhere (where?) in freerdp api to pass data into freerdp
  3. add write callback somewhere (where?) in freerdp api to pass data from freerdp

currently i figured what low level BIO based io done in "transport_read_layer" and "transport_write" (correct me if i am wrong)

open questions:

  1. where to place new api call and callback ?
  2. is any low level io implementation exists in addition to "transport_read_layer" and "transport_write" ?

sss123next avatar Jun 18 '20 20:06 sss123next

i guess best place for callback definition is "rdp_update" struct, i think i will add something like "rdp_io_update" inside it, and in this "rdp_io_update" i will add something like "WriteBack" ? or just "Write" ?

sss123next avatar Jun 20 '20 04:06 sss123next

i think it will be "Write" and "WriteBack" inside "rdp_io_update" and in case of usage external io, additional function will be called to initialize this "Write" to default implementation, i guess...

sss123next avatar Jun 21 '20 14:06 sss123next

This issue is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Oct 07 '24 03:10 github-actions[bot]

This issue was closed because it has been inactive for 14 days since being marked as stale.

github-actions[bot] avatar Nov 07 '24 03:11 github-actions[bot]

@sss123next added https://github.com/FreeRDP/FreeRDP/pull/10679 for client side

akallabeth avatar Nov 07 '24 07:11 akallabeth