ReactiveExtensionClient的listAll方法兼容sort为null的情况

longjuan commented 5 days ago

系统信息

外部访问地址:
启动时间: 2024-06-30 15:44
版本: 2.17.0-SNAPSHOT
构建时间: 2024-06-30 15:44
Git Commit: 9410006
Java: Java(TM) SE Runtime Environment / 17.0.5+9-LTS-191
数据库: H2 / 2.2.224
操作系统: Windows 10 / 10.0
已激活主题: Earth 1.7.1
已启动插件:
- 对象存储（Amazon S3 协议） 1.9.0-SNAPSHOT

使用的哪种方式运行？

Source Code

发生了什么？

https://github.com/halo-dev/halo/blob/9410006659e08367f66c03d60e60a7e817ea91b8/application/src/main/java/run/halo/app/extension/ReactiveExtensionClientImpl.java#L114 s3插件调用listAll方法时，sort参数传的时null，在旧版本halo中时可以运行的，2.17不行了。虽然可以通过修改s3代码实现兼容新版本，但是不确定其他插件是否有这种写法。希望希望能兼容此类情形，同时对其他接口进行排查。

相关日志输出

java.lang.NullPointerException: Cannot invoke "org.springframework.data.domain.Sort.iterator()" because "sort" is null
    at run.halo.app.extension.index.IndexedQueryEngineImpl.getFieldNamesUsedInListOptions(IndexedQueryEngineImpl.java:150) ~[main/:na]
    Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException: 
Error has been observed at the following site(s):
    *__checkpoint ⇢ Handler run.halo.s3os.S3LinkController#listObjects(String, String, String, Integer, Boolean, String) [DispatcherHandler]
    *__checkpoint ⇢ run.halo.app.console.ProxyFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ run.halo.app.console.ProxyFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ run.halo.app.security.InitializeRedirectionWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ run.halo.app.security.device.DeviceSessionFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ AuthorizationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ ExceptionTranslationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ LogoutWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ ServerRequestCacheWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ SecurityContextServerWebExchangeWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ LogoutPageGeneratingWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ AnonymousAuthenticationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ SecurityWebFilterChainProxy [DefaultWebFilterChain]
    *__checkpoint ⇢ TotpAuthenticationFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ AuthenticationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ SecurityWebFilterChainProxy [DefaultWebFilterChain]
    *__checkpoint ⇢ RememberMeAuthenticationFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ SecurityWebFilterChainProxy [DefaultWebFilterChain]
    *__checkpoint ⇢ AuthenticationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ AuthenticationWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ ReactorContextWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ CsrfWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ CorsWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ HttpHeaderWriterWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ ServerWebExchangeReactorContextWebFilter [DefaultWebFilterChain]
    *__checkpoint ⇢ org.springframework.security.web.server.WebFilterChainProxy [DefaultWebFilterChain]
    *__checkpoint ⇢ run.halo.app.webfilter.AdditionalWebFilterChainProxy [DefaultWebFilterChain]
    *__checkpoint ⇢ HTTP GET "/apis/s3os.halo.run/v1alpha1/objects/attachment-policy-nYTgM?continuationToken=&continuationObject=&pageSize=50&filePrefix=" [ExceptionHandlingWebHandler]
Original Stack Trace:
        at run.halo.app.extension.index.IndexedQueryEngineImpl.getFieldNamesUsedInListOptions(IndexedQueryEngineImpl.java:150) ~[main/:na]
        at run.halo.app.extension.index.IndexedQueryEngineImpl.doRetrieve(IndexedQueryEngineImpl.java:113) ~[main/:na]
        at run.halo.app.extension.index.IndexedQueryEngineImpl.retrieveAll(IndexedQueryEngineImpl.java:55) ~[main/:na]
        at run.halo.app.extension.ReactiveExtensionClientImpl.lambda$listAll$2(ReactiveExtensionClientImpl.java:117) ~[main/:na]
        at reactor.core.publisher.MonoSupplier$MonoSupplierSubscription.request(MonoSupplier.java:126) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoPeekTerminal$MonoTerminalPeekSubscriber.request(MonoPeekTerminal.java:139) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoFlatMapMany$FlatMapManyMain.onSubscribe(MonoFlatMapMany.java:141) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoPeekTerminal$MonoTerminalPeekSubscriber.onSubscribe(MonoPeekTerminal.java:152) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoSupplier.subscribe(MonoSupplier.java:48) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.Mono.subscribe(Mono.java:4568) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:265) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:165) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.MonoUsing$MonoUsingSubscriber.onNext(MonoUsing.java:231) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.publisher.FluxSubscribeOnCallable$CallableSubscribeOnSubscription.run(FluxSubscribeOnCallable.java:252) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68) ~[reactor-core-3.6.7.jar:3.6.7]
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28) ~[reactor-core-3.6.7.jar:3.6.7]
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[na:na]
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) ~[na:na]
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[na:na]
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[na:na]
        at java.base/java.lang.Thread.run(Thread.java:833) ~[na:na]