Authentication-Results: mail-b.sr.ht; dkim=pass header.d=cmpwn.com header.i=@cmpwn.com Received: from out1.migadu.com (out1.migadu.com [91.121.223.63]) by mail-b.sr.ht (Postfix) with ESMTPS id 179C611EFF3 for <~adnano/go-gemini-devel@lists.sr.ht>; Sun, 21 Feb 2021 13:58:06 +0000 (UTC) MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cmpwn.com; s=key1; t=1613915885; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=6b6b2MxUAppFm395zANBVooas6XvhpdJ2RJpUd4gEVE=; b=abb/7Dm20P88jL17nhJfE9XnoVWn6uKj3NhSD/BPsqBbV7QrdUY1dmk/pDX/5wrU+C6GPR NomdY+vnpIlb35tOnbOmTqgiQg6iim0uvLJzB3nCiaIV0nBwi8roVCOSP+Tk4KbNTax7aM bcWlzkyD4USDY+7xPbwBxTOoM6WGe9We9KZ5zqjsm+wPdo2KT67uijcsN1Rh2mZazVwDLA Lq0dun+q9aCWwd9Mob/CIFDkpr5b0ESDHf76hQ0GgqRnLxp63EFOL0ypums5jGJ0JzrXUJ BvHE5mJJvm9p6pz1iMu0Mnt7JOl6l0B/OsXG9w/SCSojB2oHdKrc7SAgN+7KlA== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Sun, 21 Feb 2021 08:58:02 -0500 Message-Id: X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "Drew DeVault" To: "Adnan Maolood" , <~adnano/go-gemini-devel@lists.sr.ht> Cc: Subject: Re: Seeking feedback for version v0.1.15 References: In-Reply-To: X-Migadu-Flow: FLOW_OUT X-Migadu-Auth-User: sir@cmpwn.com On Sun Feb 21, 2021 at 8:45 AM EST, Adnan Maolood wrote: > I could, however this makes creating derivative contexts more > difficult. One must call Request.Context(), and then call > Request.WithContext with the derived context, which must copy the > request. Also note that the http Handler API was in use before the > introduction of contexts, so they had to implement that API for > backwards compatibility. Since we don't have to worry about that yet, > I think we can come up with a better approach. That's a good point. Yours is probably the better design. I don't really mind having a context parameter in function signatures.