IronRDP icon indicating copy to clipboard operation
IronRDP copied to clipboard
verified publisher icon Devolutions

feat(ffi): expose API for RDCleanPath

Open irvingoujAtDevolution opened this issue 5 months ago • 5 comments

Expose RDCleanPath via FFI, enable IronRDP .NET connections via RDCleanPath.

Issue: ARC-310

irvingoujAtDevolution avatar Jul 03 '25 19:07 irvingoujAtDevolution

Please, fix the errors reported by the CI

CBenoit avatar Jul 04 '25 10:07 CBenoit

Also, ideally I would like two PRs:

  • One dedicated to exposing the RDCleanPath API, and
  • One dedicated to adding the WebSocket helper in the .NET library.

CBenoit avatar Jul 04 '25 14:07 CBenoit

Coverage Report :robot: :gear:

Past: Total lines: 28177 Covered lines: 17394 (61.73%)

New: Total lines: 28177 Covered lines: 17389 (61.71%)

Diff: -0.02%

[this comment will be updated automatically]

github-actions[bot] avatar Jul 05 '25 01:07 github-actions[bot]

Also, ideally I would like two PRs:

* One dedicated to exposing the RDCleanPath API, and

* One dedicated to adding the WebSocket helper in the .NET library.

I created this one dedicated for FFI changes, after it's merged I can rebase current PR to master, because the changes in FFI source would not compile without generated C# code. https://github.com/Devolutions/IronRDP/pull/864

irvingoujAtDevolution avatar Jul 08 '25 21:07 irvingoujAtDevolution

Also, ideally I would like two PRs:

* One dedicated to exposing the RDCleanPath API, and

* One dedicated to adding the WebSocket helper in the .NET library.

I created this one dedicated for FFI changes, after it's merged I can rebase current PR to master, because the changes in FFI source would not compile without generated C# code.

https://github.com/Devolutions/IronRDP/pull/864

Great, thank you!

CBenoit avatar Jul 09 '25 05:07 CBenoit