royalrick / weapp

微信小程序服务端 SDK (for Golang)
https://pkg.go.dev/github.com/medivhzhan/weapp/v3
Apache License 2.0
1.47k stars 281 forks source link

更新消息推送相关代码 #18

Closed HubertZhang closed 5 years ago

HubertZhang commented 5 years ago

修复AES解码中的错误

添加 Server 构造函数 去除 Server 内的 Request 和 ResponseWriter Server 实现 http.Handler 接口 修复 推送消息解密 中的错误

royalrick commented 5 years ago

感谢你的 Pull request,但是我仍然做出了一些我认为更方便使用的调整,欢迎一起探讨,谢谢:)

HubertZhang commented 5 years ago

额,ServeHTTP 是不应该返回参数的。。请查阅一下 http.handler 接口

royalrick commented 5 years ago

如果是当作 http.handler 的确实应该像你那样写,但是我每次是放到 http.handler 里面使用的,用之前还得校验 token 的有效性。

如果当成 http.handler 我们应该加入自动缓存 token 的功能。

如果需要,我们还可以再另外写一个函数来实现 http.handler 来方便需要的用户。

你觉得呢 :)

HubertZhang commented 5 years ago

OK 不如把 ServeHTTP 改个名好了🤔,避免和 interface 混淆

royalrick commented 5 years ago

好的👌