Alright, I&#39;ll make a small patch that adds 2 fields to SetCookie:<div>setCookieMaxAge :: Maybe DiffTime</div><div>setCookieSecureOnly :: Bool</div><div><br></div><div>I&#39;ve also gotten started on those cookie functions. I&#39;m currently writing tests for them.</div>

<div><br></div><div>@Chris: The best advice I can give is that Chrome (what I&#39;m using as a source on all this) has the data baked into a .cc file. However, they have directions in a README and a script which will parse the list and generate that source file. I recommend doing this. That way, the Haskell module would have 2 source files: one file that reads the list and generates the second file, which is a very large source file that contains each element in the list. The list should export `elem`-type queries. I&#39;m not quite sure how to handle wildcards that appear in the list - that part is up to you. Thanks for helping out with this :]</div>

<div><br></div><div>--Myles<br><br><div class="gmail_quote">On Thu, Feb 2, 2012 at 10:53 PM, Michael Snoyman <span dir="ltr">&lt;<a href="mailto:michael@snoyman.com">michael@snoyman.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Looks good to me too. I agree with Aristid: let&#39;s make the change to<br>
cookie itself. Do you want to send a pull request? I&#39;m also<br>
considering making the SetCookie constructor hidden like we have for<br>
Request, so that if in the future we realize we need to add some other<br>
settings, it doesn&#39;t break the API.<br>
<br>
Chris: I would recommend compiling it into the module. Best bet would<br>
likely being converting the source file to Haskell source.<br>
<br>
Michael<br>
<br>
On Fri, Feb 3, 2012 at 6:32 AM, Myles C. Maxfield<br>
<div class="HOEnZb"><div class="h5">&lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt; wrote:<br>
&gt; Alright. After reading the spec, I have these questions / concerns:<br>
&gt;<br>
&gt; The spec supports the &quot;Max-Age&quot; cookie attribute, which Web.Cookies doesn&#39;t.<br>
&gt;<br>
&gt; I see two possible solutions to this. The first is to have parseSetCookie<br>
&gt; take a UTCTime as an argument which will represent the current time so it<br>
&gt; can populate the setCookieExpires field by adding the Max-Age attribute to<br>
&gt; the current time. Alternatively, that function can return an IO SetCookie so<br>
&gt; it can ask for the current time by itself (which I think is inferior to<br>
&gt; taking the current time as an argument). Note that the spec says to prefer<br>
&gt; Max-Age over Expires.<br>
&gt; Add a field to SetCookie of type Maybe DiffTime which represents the Max-Age<br>
&gt; attribute<br>
&gt;<br>
&gt; Cookie code should be aware of the Public Suffix List as a part of its<br>
&gt; domain verification. The cookie code only needs to be able to tell if a<br>
&gt; specific string is in the list (W.Ascii -&gt; Bool)<br>
&gt;<br>
&gt; I propose making an entirely unrelated package, public-suffix-list, with a<br>
&gt; module Network.PublicSuffixList, which will expose this function, as well as<br>
&gt; functions about parsing the list itself. Thoughts?<br>
&gt;<br>
&gt; Web.Cookie doesn&#39;t have a &quot;secure-only&quot; attribute. Adding one in is<br>
&gt; straightforward enough.<br>
&gt; The spec describes cookies as a property of HTTP, not of the World Wide Web.<br>
&gt; Perhaps &quot;Web.Cookie&quot; should be renamed? Just a thought; it doesn&#39;t really<br>
&gt; matter to me.<br>
&gt;<br>
&gt; As for Network.HTTP.Conduit.Cookie, the spec describes in section 5.3<br>
&gt; &quot;Storage Model&quot; what fields a Cookie has. Here is my proposal for the<br>
&gt; functions it will expose:<br>
&gt;<br>
&gt; receiveSetCookie :: SetCookie -&gt; Req.Request m -&gt; UTCTime -&gt; Bool -&gt;<br>
&gt; CookieJar -&gt; CookieJar<br>
&gt;<br>
&gt; Runs the algorithm described in section 5.3 &quot;Storage Model&quot;<br>
&gt; The UTCTime is the current-time, the Bool is whether or not the caller is an<br>
&gt; HTTP-based API (as opposed to JavaScript or anything else)<br>
&gt;<br>
&gt; updateCookieJar :: Res.Response a -&gt; Req.Request m -&gt; UTCTime -&gt; CookieJar<br>
&gt; -&gt; (CookieJar, Res.Response a)<br>
&gt;<br>
&gt; Applies &quot;receiveSetCookie&quot; to a Response. The output CookieJar is stripped<br>
&gt; of any Set-Cookie headers.<br>
&gt; Specifies &quot;True&quot; for the Bool in receiveSetCookie<br>
&gt;<br>
&gt; computeCookieString :: Req.Request m -&gt; CookieJar -&gt; UTCTime -&gt; Bool -&gt;<br>
&gt; (W.Ascii, CookieJar)<br>
&gt;<br>
&gt; Runs the algorithm described in section 5.4 &quot;The Cookie Header&quot;<br>
&gt; The UTCTime and Bool are the same as in receiveSetCookie<br>
&gt;<br>
&gt; insertCookiesIntoRequest :: Req.Request m -&gt; CookieJar -&gt; UTCTime -&gt;<br>
&gt; (Req.Request m, CookieJar)<br>
&gt;<br>
&gt; Applies &quot;computeCookieString&quot; to a Request. The output cookie jar has<br>
&gt; updated last-accessed-times.<br>
&gt; Specifies &quot;True&quot; for the Bool in computeCookieString<br>
&gt;<br>
&gt; evictExpiredCookies :: CookieJar -&gt; UTCTime -&gt; CookieJar<br>
&gt;<br>
&gt; Runs the algorithm described in the last part of section 5.3 &quot;Storage Model&quot;<br>
&gt;<br>
&gt; This will make the relevant part of &#39;http&#39; look like:<br>
&gt;<br>
&gt;     go count req&#39;&#39; cookie_jar&#39;&#39; = do<br>
&gt;         now &lt;- liftIO $ getCurrentTime<br>
&gt;         let (req&#39;, cookie_jar&#39;) = insertCookiesIntoRequest req&#39;&#39;<br>
&gt; (evictExpiredCookies cookie_jar&#39;&#39; now) now<br>
&gt;         res&#39; &lt;- httpRaw req&#39; manager<br>
&gt;         let (cookie_jar, res) = updateCookieJar res&#39; req&#39; now cookie_jar&#39;<br>
&gt;         case getRedirectedRequest req&#39; (responseHeaders res) (W.statusCode<br>
&gt; (statusCode res)) of<br>
&gt;             Just req -&gt; go (count - 1) req cookie_jar<br>
&gt;             Nothing -&gt; return res<br>
&gt;<br>
&gt; I plan to not allow for a user-supplied cookieFilter function. If they want<br>
&gt; that functionality, they can re-implement the redirection-following logic.<br>
&gt;<br>
&gt; Any thoughts on any of this?<br>
&gt;<br>
&gt; Thanks,<br>
&gt; Myles<br>
&gt;<br>
&gt; On Wed, Feb 1, 2012 at 5:19 PM, Myles C. Maxfield &lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt;<br>
&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Nope. I&#39;m not. The RFC is very explicit about how to handle cookies. As<br>
&gt;&gt; soon as I&#39;m finished making sense of it (in terms of Haskell) I&#39;ll send<br>
&gt;&gt; another proposal email.<br>
&gt;&gt;<br>
&gt;&gt; On Feb 1, 2012 3:25 AM, &quot;Michael Snoyman&quot; &lt;<a href="mailto:michael@snoyman.com">michael@snoyman.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; You mean you&#39;re *not* making this proposal?<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Wed, Feb 1, 2012 at 7:30 AM, Myles C. Maxfield<br>
&gt;&gt;&gt; &lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt; &gt; Well, this is embarrassing. Please disregard my previous email. I<br>
&gt;&gt;&gt; &gt; should<br>
&gt;&gt;&gt; &gt; learn to read the RFC *before* submitting proposals.<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt; --Myles<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt; On Tue, Jan 31, 2012 at 6:37 PM, Myles C. Maxfield<br>
&gt;&gt;&gt; &gt; &lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Here are my initial ideas about supporting cookies. Note that I&#39;m<br>
&gt;&gt;&gt; &gt;&gt; using<br>
&gt;&gt;&gt; &gt;&gt; Chrome for ideas since it&#39;s open source.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Network/HTTP/Conduit/Cookies.hs file<br>
&gt;&gt;&gt; &gt;&gt; Exporting the following symbols:<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; type StuffedCookie = SetCookie<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; A regular SetCookie can have Nothing for its Domain and Path<br>
&gt;&gt;&gt; &gt;&gt; attributes. A<br>
&gt;&gt;&gt; &gt;&gt; StuffedCookie has to have these fields set.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; type CookieJar = [StuffedCookie]<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Chrome&#39;s cookie jar is implemented as (the C++ equivalent of) Map<br>
&gt;&gt;&gt; &gt;&gt; W.Ascii<br>
&gt;&gt;&gt; &gt;&gt; StuffedCookie. The key is the &quot;eTLD+1&quot; of the domain, so lookups for<br>
&gt;&gt;&gt; &gt;&gt; all<br>
&gt;&gt;&gt; &gt;&gt; cookies for a given domain are fast.<br>
&gt;&gt;&gt; &gt;&gt; I think I&#39;ll stay with just a list of StuffedCookies just to keep it<br>
&gt;&gt;&gt; &gt;&gt; simple. Perhaps a later revision can implement the faster map.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; getRelevantCookies :: Request m -&gt; CookieJar -&gt; UTCTime -&gt; (CookieJar,<br>
&gt;&gt;&gt; &gt;&gt; Cookies)<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Gets all the cookies from the cookie jar that should be set for the<br>
&gt;&gt;&gt; &gt;&gt; given<br>
&gt;&gt;&gt; &gt;&gt; Request.<br>
&gt;&gt;&gt; &gt;&gt; The time argument is whatever &quot;now&quot; is (it&#39;s pulled out of the<br>
&gt;&gt;&gt; &gt;&gt; function so<br>
&gt;&gt;&gt; &gt;&gt; the function can remain pure and easily testable)<br>
&gt;&gt;&gt; &gt;&gt; The function will also remove expired cookies from the cookie jar<br>
&gt;&gt;&gt; &gt;&gt; (given<br>
&gt;&gt;&gt; &gt;&gt; what &quot;now&quot; is) and return the filtered cookie jar<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; putRelevantCookies :: Request m -&gt; CookieJar -&gt; [StuffedCookie] -&gt;<br>
&gt;&gt;&gt; &gt;&gt; CookieJar<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Insert cookies from a server response into the cookie jar.<br>
&gt;&gt;&gt; &gt;&gt; The first argument is only used for checking to see which cookies are<br>
&gt;&gt;&gt; &gt;&gt; valid (which cookies match the requested domain, etc, so <a href="http://site1.com" target="_blank">site1.com</a><br>
&gt;&gt;&gt; &gt;&gt; can&#39;t set<br>
&gt;&gt;&gt; &gt;&gt; a cookie for <a href="http://site2.com" target="_blank">site2.com</a>)<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; stuffCookie :: Request m -&gt; SetCookie -&gt; StuffedCookie<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; If the SetCookie&#39;s fields are Nothing, fill them in given the Request<br>
&gt;&gt;&gt; &gt;&gt; from<br>
&gt;&gt;&gt; &gt;&gt; which it originated<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; getCookies :: Response a -&gt; ([SetCookie], Response a)<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Pull cookies out of a server response. Return the response with the<br>
&gt;&gt;&gt; &gt;&gt; Set-Cookie headers filtered out<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; putCookies :: Request a -&gt; Cookies -&gt; Request a<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; A wrapper around renderCookies. Inserts some cookies into a request.<br>
&gt;&gt;&gt; &gt;&gt; Doesn&#39;t overwrite cookies that are already set in the request<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; These functions will be exported from Network.HTTP.Conduit as well, so<br>
&gt;&gt;&gt; &gt;&gt; callers can use them to re-implement redirection chains<br>
&gt;&gt;&gt; &gt;&gt; I won&#39;t implement a cookie filtering function (like what<br>
&gt;&gt;&gt; &gt;&gt; Network.Browser<br>
&gt;&gt;&gt; &gt;&gt; has)<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; If you want to have arbitrary handling of cookies, re-implement<br>
&gt;&gt;&gt; &gt;&gt; redirection following. It&#39;s not very difficult if you use the API<br>
&gt;&gt;&gt; &gt;&gt; provided,<br>
&gt;&gt;&gt; &gt;&gt; and the &#39;http&#39; function is open source so you can use that as a<br>
&gt;&gt;&gt; &gt;&gt; reference.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; I will implement the functions according to RFC 6265<br>
&gt;&gt;&gt; &gt;&gt; I will also need to write the following functions. Should they also be<br>
&gt;&gt;&gt; &gt;&gt; exported?<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; canonicalizeDomain :: W.Ascii -&gt; W.Ascii<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; turns &quot;..a.b.c..d.com...&quot; to &quot;<a href="http://a.b.c.d.com" target="_blank">a.b.c.d.com</a>&quot;<br>
&gt;&gt;&gt; &gt;&gt; Technically necessary for domain matching (Chrome does it)<br>
&gt;&gt;&gt; &gt;&gt; Perhaps unnecessary for a first pass? Perhaps we can trust users for<br>
&gt;&gt;&gt; &gt;&gt; now?<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; domainMatches :: W.Ascii -&gt; W.Ascii -&gt; Maybe W.Ascii<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Does the first domain match against the second domain?<br>
&gt;&gt;&gt; &gt;&gt; If so, return the prefix of the first that isn&#39;t in the second<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; pathMatches :: W.Ascii -&gt; W.Ascii -&gt; Bool<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Do the paths match?<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; In order to implement domain matching, I have to have knowledge of<br>
&gt;&gt;&gt; &gt;&gt; the Public Suffix List so I know that <a href="http://sub1.sub2.pvt.k12.wy.us" target="_blank">sub1.sub2.pvt.k12.wy.us</a> can set<br>
&gt;&gt;&gt; &gt;&gt; a<br>
&gt;&gt;&gt; &gt;&gt; cookie for <a href="http://sub2.pvt.k12.wy.us" target="_blank">sub2.pvt.k12.wy.us</a> but not for <a href="http://k12.wy.us" target="_blank">k12.wy.us</a> (because<br>
&gt;&gt;&gt; &gt;&gt; <a href="http://pvt.k12.wy.us" target="_blank">pvt.k12.wy.us</a><br>
&gt;&gt;&gt; &gt;&gt; is a &quot;suffix&quot;). There are a variety of ways to implement this.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; As far as I can tell, Chrome does it by using a script (which a human<br>
&gt;&gt;&gt; &gt;&gt; periodically runs) which parses the list at creates a .cc file that is<br>
&gt;&gt;&gt; &gt;&gt; included in the build.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; I might be wrong about the execution of the script; it might be a<br>
&gt;&gt;&gt; &gt;&gt; build<br>
&gt;&gt;&gt; &gt;&gt; step. If it is a build step, however, it is suspicious that a build<br>
&gt;&gt;&gt; &gt;&gt; target<br>
&gt;&gt;&gt; &gt;&gt; would try to download a file...<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Any more elegant ideas?<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Feedback on any/all of the above would be very helpful before I go off<br>
&gt;&gt;&gt; &gt;&gt; into the weeds on this project.<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; Thanks,<br>
&gt;&gt;&gt; &gt;&gt; Myles C. Maxfield<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt; On Sat, Jan 28, 2012 at 8:17 PM, Michael Snoyman &lt;<a href="mailto:michael@snoyman.com">michael@snoyman.com</a>&gt;<br>
&gt;&gt;&gt; &gt;&gt; wrote:<br>
&gt;&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; Thanks, looks great! I&#39;ve merged it into the Github tree.<br>
&gt;&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; On Sat, Jan 28, 2012 at 8:36 PM, Myles C. Maxfield<br>
&gt;&gt;&gt; &gt;&gt;&gt; &lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; Ah, yes, you&#39;re completely right. I completely agree that moving<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; the<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; function into the Maybe monad increases readability. This kind of<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; function<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; is what the Maybe monad was designed for.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; Here is a revised patch.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; On Sat, Jan 28, 2012 at 8:28 AM, Michael Snoyman<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; &lt;<a href="mailto:michael@snoyman.com">michael@snoyman.com</a>&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt; wrote:<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; On Sat, Jan 28, 2012 at 1:20 AM, Myles C. Maxfield<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &lt;<a href="mailto:myles.maxfield@gmail.com">myles.maxfield@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; the fromJust should never fail, beceause of the guard statement:<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt;     | 300 &lt;= code &amp;&amp; code &lt; 400 &amp;&amp; isJust l&#39;&#39; &amp;&amp; isJust l&#39; =<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; Just $<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; req<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; Because of the order of the &amp;&amp; operators, it will only evaluate<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; fromJust<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; after it makes sure that the argument isJust. That function in<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; particular<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; shouldn&#39;t throw any exceptions - it should only return Nothing.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; Knowing that, I don&#39;t quite think I understand what your concern<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; is.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; Can<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; you<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; &gt; elaborate?<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; You&#39;re right, but I had to squint really hard to prove to myself<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; that<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; you&#39;re right. That&#39;s the kind of code that could easily be broken<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; in<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; future updates by an unwitting maintainer (e.g., me). To protect<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; the<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; world from me, I&#39;d prefer if the code didn&#39;t have the fromJust.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; This<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; might be a good place to leverage the Monad instance of Maybe.<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;&gt; Michael<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;&gt;<br>
&gt;&gt;&gt; &gt;<br>
&gt;<br>
&gt;<br>
</div></div></blockquote></div><br></div>