bug(JsonStringLocalizer): add type check prevent endless loop

[izanhzh]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

GetStringSafely和GetAllStrings这两个方法中，存在个死循环的可能：

Expected Behavior

在上述第2步中，增加判断排除自身避免死循环

Interactive render mode

Interactive Server (Interactive server-side rendering (interactive SSR) using Blazor Server)

Steps To Reproduce

xxx

Exceptions (if any)

.NET Version

dotnet all version

Anything else?

No response

[bb-auto]

@izanhzh Thank you for filing this issue. In order for us to investigate this issue, please provide a minimalistic repro project that illustrates the problem.

[ArgoZhang]

@izanhzh 死循环的根本原因我个人认为是代码导致的

public IStringLocalizer Create(Type resourceSource)
{
    var stringLocalizerFactorys = _serviceProvider.GetServices<IStringLocalizerFactory>();
    IStringLocalizerFactory stringLocalizerFactory;
    if (resourceSource == typeof(Foo))
    {
        stringLocalizerFactory = stringLocalizerFactorys.Single(s => s.GetType().Name == "JsonStringLocalizerFactory");
    }
    else
    {
        stringLocalizerFactory = _serviceProvider.GetServices<IStringLocalizerFactory>().Single(s => s is MockLocalizerFactory);
    }
    return stringLocalizerFactory.Create(resourceSource);
}

这里的逻辑有很大问题

BootstrapBlazor 组件内部已经找到合适并且正确的 IStringLocalizerFactory 服务，这块代码里不应该去取一个错误的服务

[izanhzh]

@ArgoZhang 我认为JsonStringLocalizer应该做好防御，不能让其他人能够将从外部返回一个JsonStringLocalizer实例给到GetStringSafely和GetAllStrings这两个方法中，别人集成项目中，可能会使用自己定义的IStringLocalizerFactory，然后基于一些情况让其返回BB默认的JsonStringLocalizer，其他情况返回其他的xxxStringLocalizer，上面那个代码只是一种方式，比如我其他的xxxStringLocalizer不通过IOC容器注入，容器中只有BB的JsonStringLocalizer，然后我代码写成某些情况从容器中获取，某些情况通过一些其他手段构造StringLocalizer
当然这也不算严格意义上的BUG，应该调整为是feat

[ArgoZhang]

这里没办法做防护的，我认为应该做个检查，发现死循环了报错比较合理。因为你不知道外面代码如何写，那我们就没有办法做防护。

也不行。死循环都无法检查。你的这个单元测试本身就有问题。并不能代表所有问题

你这个代码是服务里再拿一个不正确的服务，所以导致了死循环

[ArgoZhang]

@izanhzh 应该找到一个合理的解决方案。不能被你的单元测试误导了

[izanhzh]

不是很明白，Utility.GetStringLocalizerFromService 这里不应该杜绝掉JsonStringLocalizer返回么，我认为还是很有必要的，这里防护一下，JsonStringLocalizer就不会陷入循环了，如果Utility.GetStringLocalizerFromService 返回的是JsonStringLocalizer就陷入循环了

[izanhzh]

Utility.GetStringLocalizerFromService 过滤掉返回JsonStringLocalizer，接下里就已经是JsonStringLocalizer在处理了，这是正确的预期效果，Utility.GetStringLocalizerFromService 本就不应该有返回JsonStringLocalizer的可能

[ArgoZhang]

@izanhzh 这个对。所以我刚才说应该抛异常，我改一下。你看看是否更合理

[izanhzh]

@ArgoZhang 我还是觉得没必要抛异常，既然外部返回的就是JsonStringLocalizer，那咱就用JsonStringLocalizer给它处理不就行了，没必要报错呀，这不就符合外部想要的效果了么，这个更灵活呀 😂

[izanhzh]

[ArgoZhang]

@izanhzh 行吧。你这例子也是奇葩写法。这么折腾一圈还搞个死循环，我内部还要判断一下，为啥啊？为啥非要拿我内部的服务搞啊。所以我想抛异常