建议：期望能拿到当前输出日志的level
jiladahe1997 opened this issue · comments
问题描述：日志需要上传到云端，云端期望能对日志做分级(fatal/error/warn/debug)，因此建议的json格式是这样：
{ logcontent: "这是一条日志", level: "debug" }
我个人觉得好像有一丝道理。。
可以在elog_port_output函数中增加增加一个level参数。
// 现在是这样
void elog_port_output(const char *log, size_t size);
// 期望变成这样
void elog_port_output(const char *log, size_t size, level_t level);
这样的话，我就可以这样进行日志上报：
void elog_port_output(const char *log, size_t size, level_t level) {
char log_str[1024] = {0};
sprintf(log_str, "{ \"logconten\"t: "%s", \"level\": "%s" }", log, level);
upload_log(log_str);
}
但是上述的方法有一个弊端：上报的日志包含了一些颜色、对齐等字符。通常上报或者保存到持久存储的日志是不希望有这些字符。另外，如果不需要上报日志，level参数也用不上。
所以建议增加一个“生命周期钩子函数”，每条日志调用 elog_port_output 输出后都会去调用这个函数，函数的参数是原本的日志内容（没有添加颜色或者其他信息）,用户可以自行实现这个函数，从而自定义额外的日志保存、上传等操作。
void elog_output_hook(const char *log,level_t level);
挺好的建议，但可能得下一个大版本改动才能够去实现了
如果你着急使用的话，建议参考下我之前做的 ulog ，里面的底层输出接口应该符合你的要求