dependency resolution on a non-JS thread fails with 0x80131509 instead of falling back
還沒有人認領這個 Issue。
評估
- 難度
- 3/5
- 預估耗時
- 1-2 天
- 新手友好度
- 70/100
研究方向
Start with ManagedHost.OnResolvingAssembly in src/NodeApi.DotNetHost/ManagedHost.cs, especially the FileNotFoundException handler and adjacent-directory fallback. Read how the resolving event is emitted and how JSValueScope.CurrentOrNull indicates whether a JS scope is active. Reproduce the off-thread case using the linked minimal repo; done when assembly resolution reaches the adjacent-directory fallback without throwing on a thread with no JS scope.
由索引模型根據 Issue 內容生成。
描述
Environment
- node-api-dotnet /
Microsoft.JavaScript.NodeApi(.*): 0.9.27 - Node.js: v24.19.0 (Windows x64)
- .NET runtime loaded by the host: 8.0.29
Summary
When a dependency assembly is first resolved on a thread that has no JS value scope (a .NET thread-pool
thread in the example below), ManagedHost.OnResolvingAssembly throws JSInvalidThreadAccessException while
trying to emit the resolving event.
The runtime exposes that as a FileLoadException:
System.IO.FileLoadException: Could not load file or assembly 'Newtonsoft.Json, Version=13.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'. An operation is not legal in the current state. (0x80131509)
---> Microsoft.JavaScript.NodeApi.JSInvalidThreadAccessException: There is no active JS value scope.
Current thread: #4 ".NET TP Worker". Consider using the synchronization context to switch to the JS thread.
Note this is 0x80131509 (COR_E_INVALIDOPERATION), not 0x80070002 (file not found): in the repro the assembly is next to the module, so the message points away from the cause.
It also throws before the loader's next step, which looks for the assembly in the same directory as any already-loaded assemblies: see https://github.com/microsoft/node-api-dotnet/blob/main/src/NodeApi.DotNetHost/ManagedHost.cs#L280-L316
try
{
assembly = LoadAssembly(assemblyName, allowNativeLibrary: false);
}
catch (FileNotFoundException)
{
// not found in the system directories.
// Emit a resolving event to allow listeners to load the assembly.
Emit( ResolvingEventName, assemblyName, assemblyVersion!,
new JSFunction(ResolveAssembly) //////// may throw here: building the JSValue/JSFunction arguments requires a JS scope
);
_loadedAssembliesByName.TryGetValue(assemblyName, out assembly);
}
/////////// not reached when the Emit above throws
if (assembly == null)
{
// The dependency assembly was not resolved by an event-handler.
// Look for it in the same directory as any already-loaded assemblies.
// ...
}
Minimal reproduction
see the minimal repo that reproduces
Trace evidence (NODE_API_TRACE_HOST=1)
resolve assembly on .NET Task Pool:
> ManagedHost.OnResolvingAssembly(Newtonsoft.Json, 13.0.0.0)
> ManagedHost.LoadAssembly(Newtonsoft.Json) <-- (1) framework dirs
[repro] ... FileLoadException 0x80131509 / JSInvalidThreadAccessException ".NET TP Worker"
<-- no `ManagedHost.LoadAssembly(<path>)` line, no `not found at ...` line: (3) never ran
resolve assembly on JS Scope:
> ManagedHost.OnResolvingAssembly(Newtonsoft.Json, 13.0.0.0)
> ManagedHost.LoadAssembly(Newtonsoft.Json)
> ManagedHost.LoadAssembly(<module dir>\Newtonsoft.Json.dll) <-- (3) adjacent-directory fallback
< ManagedHost.OnResolvingAssembly(Newtonsoft.Json) => <module dir>\Newtonsoft.Json.dll
Impact
- the adjacent-directory fallback never runs when resolution happens off the JS thread, so a dependency that would have resolved fails
- The error message is misleading
Suggested
Skip the notification when there is no JS scope. Since there's no JSValueScope, there's no need to report to JS
catch (FileNotFoundException)
{
// guard: use the existing internal api
if (JSValueScope.CurrentOrNull is not null)
{
Emit(ResolvingEventName, assemblyName, assemblyVersion!, new JSFunction(ResolveAssembly));
_loadedAssembliesByName.TryGetValue(assemblyName, out assembly);
}
}
With this, it will fallback to the next step.
Further Requests
Orthogonal to this report: the host's load context is private and its resolution policy is fixed, so a consumer
cannot use his own AssemblyLoadContext. It's better to allow developer to custom the ALC
- 主要語言
- C#
- 星號
- 787
- 分支
- 80
- PR 合併指標
- 30 天內沒有已合併 PR
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue,再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫,在一個分支上完成修改。
- 送出 Pull Request,並在描述裡引用這個 Issue 編號。
microsoft/node-api-dotnet 的其他 Issue
-
難度 1/5 1-3 小時 新手友好度 88/100
microsoft/node-api-dotnet#484 ·
-
Embedding args occasionally get trailing garbage on .NET (missing null terminator on a pooled buffer)可能已有人在做 @tolmachev-pravo 於 92 天前認領。 未關閉
難度 2/5 1-3 小時 新手友好度 82/100
microsoft/node-api-dotnet#481 ·
-
JSError.ThrowError(Exception) aborts the process when a Worker terminates during a .NET call可能已有人在做 @GalaxiasKyklos 於 1 天前認領。 未關閉
難度 4/5 3-5 天 新手友好度 18/100
microsoft/node-api-dotnet#505 ·
-
難度 3/5 1-2 天 新手友好度 82/100
microsoft/node-api-dotnet#502 ·
-
難度 5/5 一週以上 新手友好度 35/100
microsoft/node-api-dotnet#486 · 1 則留言 ·
查看 microsoft/node-api-dotnet 的全部 Issue
相似的 Issue
-
area-signalr
難度 2/5 1-3 小時 新手友好度 75/100
dotnet/aspnetcore#69842 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 64/100
維護者通常 2 天內回覆
-
bug needs-triage
難度 2/5 1 小時以內 新手友好度 78/100
microsoft/microsoft-ui-reactor#1373 ·
維護者通常 1 天內回覆
-
Client customer-reported KeyVault needs-team-attention question Service Attention
難度 2/5 1-3 小時 新手友好度 62/100
Azure/azure-sdk-for-net#63861 · 3 則留言 · 1 個 reaction ·
維護者通常 1 天內回覆
-
:watch: Not Triaged dotnet-csharp/svc fundamentals/subsvc
難度 1/5 1 小時以內 新手友好度 85/100
維護者通常 1 天內回覆