[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, 18 Feb 2017 13:11:45 +0200 |
> 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?