search

charleslo1 / weapp-cookie

一行代码让微信、头条、百度、支付宝小程序支持 cookie，兼容 uni-app 🍪🚀 One line of code allows weapp to support cookie（wx weixin wxapp cookie）
MIT License
829 stars 98 forks source link

读取到到cookie似乎不完整 #1

Closed falseen closed 6 years ago

falseen commented 6 years ago

实际操作的时候发现cookie不完整，少了一部分。我反复查看和修改了下面的代码，最终也没找到问题所在：

    let arr = setCookieStr.split(/\s*\\;\s*/g)
    arr.forEach((item, i) => {
      let temp = item.split('=')
      if (i === 0) {
        this.key = temp[0]
        this.value = temp[1]
      } else {
        let prop = temp[0]
        let value = temp[1]

我觉得应该是这段代码的问题。

charleslo1 commented 6 years ago

解决了，谢谢反馈，确实是这块的解析代码有点问题，已使用 set-cookie-parser 库解析 Set-Cookie 字段