[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1390435 - in /subversion/trunk: ./ subversion/include/private/ subversion/libsvn_delta/ subversion/libsvn_fs_fs/ subversion/libsvn_fs_util/ subversion/libsvn_repos/ subversion/libsvn_subr/ subversion/tests/libsvn_fs/ tools/dist/

From: Stefan Sperling <stsp_at_elego.de>
Date: Thu, 27 Sep 2012 10:31:34 +0200

On Wed, Sep 26, 2012 at 06:19:10PM +0200, Lieven Govaerts wrote:
> Although I agree that a general overview would be nice, given that the
> detailed info is easily available via:
> $ svn log --c 1390435
>
> maybe it would be easier to add these log messages (or a shortened
> version) to the commit mail instead of asking the committer to copy
> all of them?
> Or via the viewvc url?

I think it's reasonable to write a summary of the implemented
changes, with details available in the individual branch commits
for those interested. Here's one example where I did this:

------------------------------------------------------------------------
r871120 | stsp | 2008-05-06 18:29:01 +0200 (Tue, 06 May 2008) | 32 lines

Merge the dont-save-plaintext-passwords-by-default branch into trunk.

Summary of the functionality implemented:

- Add a 'store-plaintext-passwords' option, which can be set to
  'yes', 'no', or 'ask' (which is the default). If the option
  is set to 'ask', ask the user before saving plaintext passwords
  to disk. This is done from a callback that should be implemented
  by all clients. The callback is called once per authentication realm.
  If a client does not implement the callback, and does not provide
  an explicit default value, we default to storing plaintext passwords.
  I don't like this, but it's a compromise because we don't really want
  the semantics of the old API to change retroactively.
  We can, however, change this default anytime by tweaking a single
  line of code, should we decide to do so. The old API has been deprecated.

- Provide an implementation of the callback for our command line
  client, which, in addition to prompting, prints a warning explaining
  that the password can only be stored unencrypted, and shows the name
  of the authentication realm in question.
  
- Authentication parameter configuration has been moved to the
  RA layer completely. For users, this means that parameters
  which used to be configured in the [auth] section in the 'config'
  file are now configured in the 'servers' file, and can be specified
  both in the [global] section, and per server group. The [auth]
  section in 'config' is still being evaluated, but is overridden
  by anything specified in 'servers'.

Please consult the log messages of the branch for a detailed list
of all changes.

------------------------------------------------------------------------
Received on 2012-09-27 10:34:39 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.