[Wasm] Satellite assemblies for localization are not used

[jeromelaban]

Description

When adding .withConfig({ loadAllSatelliteResources: true }), satellite assemblies are all downloaded and loaded by the runtime, but using resources from those satellites resources does not work properly.

Reproduction Steps

The following:

var r = new System.Resources.ResourceManager("FxResources.System.Web.Services.Description.SR", typeof(System.Web.Services.Description.Binding).Assembly);
Console.WriteLine($"Res(en): {r.GetString("WebDescriptionMissing", new CultureInfo("en-US"))}");
Console.WriteLine($"Res(fr): {r.GetString("WebDescriptionMissing", new CultureInfo("fr"))}");

Repro: 107742-testresources01.zip

Expected behavior

The second line should return french.

Actual behavior

Res(en): Cannot find definition for {0}.  Service Description with namespace {1} is missing.
Res(fr): Cannot find definition for {0}.  Service Description with namespace {1} is missing.

Regression?

Feels like it is, net8 got it available.

Known Workarounds

None.

Configuration

.NET 9 RC1

Other information

No response

[lewing]

should be fixed in #107398

[dotnet-policy-service]

Tagging subscribers to this area: @dotnet/area-system-resources
See info in area-owners.md if you want to be subscribed.

[dotnet-policy-service]

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

[jeromelaban]

The issue is still present with today's rc2:

Installed Workload Id      Manifest Version                      Installation Source
------------------------------------------------------------------------------------
wasm-experimental          9.0.0-rc.2.24461.16/9.0.100-rc.2      SDK 9.0.100-rc.2
wasm-tools                 9.0.0-rc.2.24461.16/9.0.100-rc.2      SDK 9.0.100-rc.2

[lewing]

The issue is still present with today's rc2:

Installed Workload Id      Manifest Version                      Installation Source
------------------------------------------------------------------------------------
wasm-experimental          9.0.0-rc.2.24461.16/9.0.100-rc.2      SDK 9.0.100-rc.2
wasm-tools                 9.0.0-rc.2.24461.16/9.0.100-rc.2      SDK 9.0.100-rc.2

fix just merged, tomorrow's sdk should have it

[jeromelaban]

@lewing I've tried RC2 again (9.0.100-rc.2.24468.2-win-x64)

Installed Workload Id      Manifest Version                     Installation Source
-----------------------------------------------------------------------------------
wasm-experimental          9.0.0-rc.2.24468.6/9.0.100-rc.2      SDK 9.0.100-rc.2
wasm-tools                 9.0.0-rc.2.24468.6/9.0.100-rc.2      SDK 9.0.100-rc.2

And the issue is still present.

[maraf]

I'll take a look

[maraf]

It works with 9.0.100-rc.2.24473.20. Can you please validate it with this version?
I'm sorry, error in my setup. The problem still exists. It's related only to loadAllSatelliteResources=true. It can be bypassed by disabling fingerprinting (WasmFingerprintAssets=false) atm.

@jeromelaban Are you loading all satellites in production? It does load all cultures

[jeromelaban]

@maraf yes, it's needed in production cases. Wouldn't it also affect partial eager loading of some languages?

[maraf]

No, it uses a different path in codebase. We have test coverage for his path. The loadAllSatelliteResources=true option was added mostly for runtime tests.