[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Refactor digest authentication in url-auth
From: |
Eli Zaretskii |
Subject: |
Re: Refactor digest authentication in url-auth |
Date: |
Sat, 25 Feb 2017 10:54:27 +0200 |
Ping!
> Date: Sat, 18 Feb 2017 13:11:45 +0200
> From: Eli Zaretskii <address@hidden>
> Cc: address@hidden
>
> > From: Jarno Malmari <address@hidden>
> > Cc: address@hidden
> > Date: Tue, 14 Feb 2017 23:12:55 +0200
> >
> > Here's a revised patch set for the earlier digest auth work.
> >
> > * Changed comments to follow gnu coding standards: two spaces between
> > sentences, capital letter at beginning, period at the end, and the
> > like
> >
> > * Removed bunch of references to RFC 2617 from docstrings. Still to
> > understand what's being done requires you to pick that up, but
> > mentioning it repeatedly doesn't help too much.
> >
> > * Changed the format of commit messages.
> >
> > * There were talk about what to call the "path component of an URI".
> > Is it file-name part, directory, path, or URI path. I tried avoiding
> > mentioning it but a few references remains and I ended up choosing
> > "directory" since that's what's used in functions used here, namely:
> >
> > (url-file-directory (url-filename url))
> >
> > The `url-file-directory' is defined in url-util with docstring:
> >
> > "Return the directory part of FILE, for a URL."
> >
> > Patches that follows:
> > [PATCH 1/2] Refactor digest authentication in url-auth
> > [PATCH 2/2] Initial implementation for HTTP Digest qop for url
>
> Thanks.
>
> Could someone familiar with url-auth.el please review these?