search

luoxiaojun1992 / laravel-tars

Tars driver for laravel.
Apache License 2.0
28 stars 9 forks source link

fix: 添加tars部署配置文件路径到env文件 #16

Closed dmf-code closed 4 years ago

dmf-code commented 4 years ago

这个配置无法实现在业务逻辑中获取配置路径

config(['tars.deploy_cfg' => $cfg]);

希望能够支持添加到 env 文件，然后业务逻辑中就能够初始化 CommunicatorConfig 类

luoxiaojun1992 commented 4 years ago

config('tars.deploy_cfg') 就是tars平台自动生成的配置文件的路径呀

dmf-code commented 4 years ago

config('tars.deploy_cfg') 就是tar平台自动生产的配置文件的路径呀

嗯嗯，但是这个不会落地，导致主进程打印会有，worker进程就会没有获取到。我调试的时候发现是不同的进程来的(用的是getmypid函数)

luoxiaojun1992 commented 4 years ago

config('tars.deploy_cfg') 就是tar平台自动生产的配置文件的路径呀

嗯嗯，但是这个不会落地，导致主进程打印会有，worker进程就会没有获取到。我调试的时候发现是不同的进程来的(用的是getmypid函数)

应该是在router中重新create app导致的，我尝试修复了下，你更新master的代码再试一下，谢谢

dmf-code commented 4 years ago

config('tars.deploy_cfg') 就是tar平台自动生产的配置文件的路径呀

嗯嗯，但是这个不会落地，导致主进程打印会有，worker进程就会没有获取到。我调试的时候发现是不同的进程来的(用的是getmypid函数)

应该是在router中重新create app导致的，我尝试修复了下，你更新master的代码再试一下，谢谢

好的

dmf-code commented 4 years ago

image image 还是没有解决 config 获取问题，而且引入了其它的 bug #17 。我是直接使用 laravel-tars-demo 的。