Skip to content

Various SDK and some API docs fixes - #7245

Closed
kc284 wants to merge 4 commits into
xapi-project:masterfrom
kc284:master
Closed

kc284 wants to merge 4 commits into
xapi-project:masterfrom
kc284:master

Conversation

@kc284

@kc284 kc284 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor
  • Efficiency improvements in HTTP.cs (these have been in XC for a while now).
    • Do not re-invoke the cancellingDelegate before throwing the exception if a cancellation has been requested.
    • Report progress update per 1 second, not 1/2.
    • Only get proxy credentials when needed.
  • Minor wording changes in API docs
  • Removed error overrides that are the same as the API errors in C#.

- Do not re-invoke the cancellingDelegate before throwing the exception if a cancellation has been requested.
- Report progress update per 1 second, not 1/2.
- Only get proxy credentials when needed.

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
@kc284
kc284 requested review from mzahid83 and xueqingz September 1, 2026 15:03
@kc284
kc284 force-pushed the master branch 4 times, most recently from 2874123 to 7658ede Compare September 9, 2026 18:13
@kc284

kc284 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

I pushed accidentally other changes that were not ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant