glynnbird commented on pull request #245: URL: https://github.com/apache/couchdb-nano/pull/245#issuecomment-748875431
@robman87 I think your solution is certainly more compact! But cookie parsing can be a more complex business - imagine someone is using to communicate with CouchDB behind a proxy or a load balancer that also sets its own cookies. It's probably best to use a "real" library to make sure that all cookies are set, replayed and expired properly if this library is going to support cookies (optionally). I've put the resultant branch into a PR here https://github.com/apache/couchdb-nano/pull/245 and I'll close this one. Many thanks for raising the problem. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
