Revert "feat: add secure cookie override to agent" (#1537)

This reverts commit 737697f.

src/agent-base.js

@@ -24,8 +24,7 @@ function Agent() {
   'key',
   'pfx',
   'cert',
-  'disableTLSCerts',
-  'sendSecureCookie'
+  'disableTLSCerts'
 ].forEach(fn => {
   // Default setting for all requests from this agent
   Agent.prototype[fn] = function(...args) {

src/node/agent.js

@@ -50,10 +50,6 @@ function Agent(options) {
     if (options.rejectUnauthorized === false) {
       this.disableTLSCerts();
     }
-
-    if (options.sendSecureCookie) {
-      this.sendSecureCookie();
-    }
   }
 }
 
@@ -80,14 +76,11 @@ Agent.prototype._saveCookies = function(res) {
  */
 
 Agent.prototype._attachCookies = function(req) {
-  const sendSecureCookie = Boolean(
-    this._defaults.find(current => current.fn === 'sendSecureCookie')
-  );
   const url = parse(req.url);
   const access = new CookieAccessInfo(
     url.hostname,
     url.pathname,
-    url.protocol === 'https:' || sendSecureCookie
+    url.protocol === 'https:'
   );
   const cookies = this.jar.getCookies(access).toValueString();
   req.cookies = cookies;

src/node/index.js

@@ -646,19 +646,6 @@ Request.prototype.disableTLSCerts = function() {
   return this;
 };
 
-/**
- * Sends secure cookies on http and https requests
- * Be warned this allows cookie hijacking
- *
- * @return {Request} for chaining
- * @api public
- */
-
-Request.prototype.sendSecureCookie = function() {
-  this._sendSecureCookie = true;
-  return this;
-};
-
 /**
  * Return an http[s] request.
  *