guqing

Results 272 comments of guqing

/kind feature /area core /area ui /priority important-longterm

允许让插件添加 SimpleUrlHandlerMapping 即可实现 websocket 功能

> 经过一段时间研究,想要在插件中实现 WebSocket 功能似乎不是定义一个 SimpleUrlHandlerMapping 那么容易。尤其是需要动态注册 WebSocketHandler,还需要对 WebSocket 接口增加权限校验,需要完整考虑后才能够正式实现。 请参考以下的示例: 在 halo 中简单写一个类用来代理插件中的 SimpleHandlerMapping 然后在插件启动时将其注册到代理的 HandlerMapping 中以便被 DispatcherHandler 使用到 ```java @Component public class PluginHandlerMappingIntegrator { @Autowired private PluginSimpleUrlHandlerMapping pluginSimpleUrlHandlerMapping; //...

Hi @lingrottin, Thanks for your contribution, please fix the code style by [Code Style](https://docs.halo.run/developer-guide/core/code-style/) and unit test cases first

> /assign > > > > 我将尝试把 IndicesService 作为 PluginApplicationContext bean 的一员,并移动到 api module 中。 建议这个先不做 先重构一下搜索引擎的扩展 目前只能添加文章 希望可以添加其他的如moments或者doc等 参考 https://github.com/halo-dev/halo/issues/5202

> Hi @guqing ,我这里有几个小问题: > > > > - 如果我们重构搜索引擎的扩展,以前的扩展(run.halo.app.search.post.PostSearchService)是否还生效呢? > > - 如果仍旧生效且重构工作不能发布在 2.13,建议先将 IndicesService 加入到 PluginApplicationContext bean 中。 > > > > 最后,我更建议提供新的扩展点来重构搜索功能。 旧的目前只有core和另外一个搜索插件在使用,等重构之后可以为那个插件提一个pr适配一下然后废弃当前的功能以新的为准 毕竟之前没有提供搜索扩展的文档后续版本变动应该是正常的, 关联 https://github.com/halo-dev/halo/issues/5184

单元测试失败了,需要修复一下

> 目前,你可以试试 [WebP 插件](https://www.halo.run/store/apps/app-fVJSY)以及对应的 [WebP 服务](https://webp.se/)来优化图片的问题。 > > 另外一个方案就是使用第三方对象存储,例如 S3。 > > /triage needs-information webp 这个插件好像不会这相册这个页面生效

/triage duplicate see also #2387