You're right, I missed that you already had that one in my eagerness to get
the path property.


On Sat, Oct 19, 2013 at 5:43 PM, Joshua Holbrook
<notifications@github.com>wrote:

> In index.js:
>
> > @@ -53,6 +53,13 @@ module.exports = function (line) {
> >      parsed[label] = field;
> >    });
> >
> > +  var matches = parsed.request.match(/([A-Z]+)\s+(\S+)\s+([A-Z]+\/[\d\.]+)/);
> > +  if (matches) {
> > +    parsed.method = matches[1];
>
> Isn't this redundant with L86? I can fix it if it is, I just want to make
> sure I understand this right.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jesusabdullah/node-clf-parser/pull/2/files#r7080610>
> .
>



-- 
Tom Boutell
P'unk Avenue
215 755 1330
punkave.com
window.punkave.com