Skip to content

Remove hard dependency on wininet.dll from mip_ClientTelemetry#1428

Open
Moyin-B wants to merge 1 commit intomip/3.9.22.3/customfrom
Moyin-B/remove-wininet-dependency
Open

Remove hard dependency on wininet.dll from mip_ClientTelemetry#1428
Moyin-B wants to merge 1 commit intomip/3.9.22.3/customfrom
Moyin-B/remove-wininet-dependency

Conversation

@Moyin-B
Copy link
Copy Markdown

@Moyin-B Moyin-B commented Apr 24, 2026

The SDK links against wininet.lib, creating a load-time dependency on wininet.dll. When customers disable WinINet on their servers, the DLL fails to load entirely, breaking MIP SDK usage even though no WinINet functions are actually called.

This change Removes the hard link-time dependency on wininet.dll from the mip_ClientTelemetry.dll. The MIP SDK never uses the built-in WinINet HTTP client it always provides its own IHttpClient implementation so this dependency is unnecessary and causes failures for customers who disable WinINet on their servers.

@Moyin-B Moyin-B requested a review from a team as a code owner April 24, 2026 09:52
@AlvaroPalmaAsteMicrosoft
Copy link
Copy Markdown

Just out of curiosity, is any of our building processes ever defining HAVE_MAT_WININET_HTTP_CLIENT? Or is it defined directly in the library in a config file?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants