API: retrieve all online users with their IPs#5776
Conversation
bff4086 to
8751821
Compare
|
为什么不给 cmdOnlineStatsIpList 加一个 -all 的选项? |
…t to support retrieving all online users
|
@Fangliding done |
|
。。。 |
|
@Fangliding why? |
|
i removed new CLI method and extended existing (d963245) as you asked... |
|
本来有一个用email query的选项 复用一下加个逻辑query全部不就行了 基础设施全在这里 结果你只是wrap了一下入口点 看起来你并没有考虑过内部的安排只是为了自己面板的需求让AI往核心堆垃圾然后打开PR |
|
至于别的 你们已经有了webhook 我没有很反对它的原因是因为它的的确确是一个非常强大的功能 你们完全可以通过它在自己的软件里构建一个更为完善的在线系统(你们甚至可以追踪具体的访问时间 访问网站次数和频率) 而不是在core再添加一套API |
|
if you meant to modify grpc method |
|
话说这个和现有 Online Map API 的区别在于? |
|
The existing Online Map API doesn't have a way to get all users with their IPs in a single call. Right now you need N+1:
The |
|
It would be nice if there were a single complete query with ips, downlink, uplink: |
|
实现上面这个 #5776 (comment) 来获取所有相关信息吧,加参数决定要不要仅 online 的、要不要流量统计,后续可继续扩展 |
Added methods to gRPC API and CLI command
Usage