fregie

Any update of this? Maybe I can help implement this feature @cad

我必须要吐槽一下，这个库的架构设计实在是太差了，把功能分成这么多包的同时缺又在实现上强耦合，最让我无法接受的是，每个包主要结构的实例化、初始化居然是通过注册方式在自己包内实现的。各个功能模块在逻辑上分的比较清晰，但是在实现上各个模块界限十分模糊。另外如果tunnel必须要知道自己上层的tunnel是什么并加以处理的话，就意味着这些tunnel之间是强耦合的，不能自由拆装的，那么抽象出来的意义就不大了。 另外目前存在的bug也还不少，关于这个库我还有一些想法，可否进一步交流一下？ 请给我发邮件: [email protected] @Loyalsoldier @p4gefau1t

建议一个方案：为目前statistic里的memory实现的Authenticator，增加一个数据持久化的interface,memory可以只用该interface{}进行数据持久化，而该interface{}可以使用mysql,sqlite等不同方式实现，非侵入，易拓展。 @Loyalsoldier

> https://github.com/p4gefau1t/trojan-go/pull/356 > > There you are. @sousa-chinensis Thank you! But this is not what I want.This mysql module is for compatible with the origin trojan.What I need is a...

Can we merge this? This bug really affected me. @p4gefau1t @Loyalsoldier

> @fregie in the sample api after adding a user, the server did not close the stream in time idk of its bug yes,you need to close it by cancel...

@aztecrabbit The author has forgoten this repo. You can use fixed version by my fork. https://github.com/fregie/trojan-go/tree/latest

https://github.com/fregie/trojan-go 可以试试我修复后的版本

https://github.com/fregie/trojan-go 可以试试我修复后的版本

需要该功能的同学可以先用我的分支构建，同时修复了一些其他bug，增加了数据持久化等新功能。 https://github.com/fregie/trojan-go/tree/latest