Gnus development mailing list
 help / color / mirror / Atom feed
From: Ted Zlatanov <tzz@lifelogs.com>
To: ding@gnus.org
Subject: Re: auth-source tokens
Date: Sun, 14 Nov 2010 18:59:11 -0600	[thread overview]
Message-ID: <87tyjjpfkw.fsf@lifelogs.com> (raw)
In-Reply-To: <87pqu795st.fsf@gmx.de>

On Sun, 14 Nov 2010 18:24:50 +0100 Michael Albinus <michael.albinus@gmx.de> wrote: 

MA> Ted Zlatanov <tzz@lifelogs.com> writes:
>> On Fri, 29 Oct 2010 10:04:59 +0200 Michael Albinus <michael.albinus@gmx.de> wrote: 
>> 
>>>> A string value is matched as a regex for the
>>>> file (netrc) backend and literally by the Secrets API.  A symbol
>>>> is matched as its string value.  All the SPEC values can be
>>>> single values or lists.
>> 
MA> Hmm, maybe we shall extend secrets.el to support also regexp search?
>> 
>> It would be nice to make the interface consistent.

MA> OK. I'll check next days whether secrets.el can be extended this
MA> way. The Secrets API doesn't support it, it checks the attribute values
MA> "... via case-sensitive string equality". That means all token must be
MA> retrieved using only attribute matches which are *not* a regex. From the
MA> resulting tokens those must be removed, whose attributes with regex do
MA> not match.

Hrrrrm.  I guess we can pass "" and then examine the results directly.
But it sucks.  OTOH I don't see a need for substring or regex searching
for secrets anyhow, now that I think about it.  Is it really necessary?

MA> I still have a comment:
...
MA> How do you handle attributes which have a regex as value in SPEC? They
MA> shouldn't be created with that value.

Good catch.  I was thinking of them in Perl terms, where a regex is a
distinct kind of object and forgot ELisp doesn't have that luxury.  So
either we go with the plan above to allow only exact matching or we
create a special kind of object that can be passed to the search
function to indicate a regex search (and said object can carry the
creation value).

My vote is to do exact matching only.  It will produce simpler code and
will be easier to understand in the manual.  If it's going to be used
all over Emacs, it should be so.

Ted




  reply	other threads:[~2010-11-15  0:59 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-25  2:59 nnimap-username Daiki Ueno
2010-10-25  6:33 ` nnimap-username Reiner Steib
2010-10-25  7:13   ` nnimap-username Daiki Ueno
2010-10-25 18:09     ` auth-source tokens (was: nnimap-username) Ted Zlatanov
2010-10-26 16:56       ` auth-source tokens Ted Zlatanov
2010-10-29  8:04         ` Michael Albinus
2010-10-29 22:15         ` Lars Magne Ingebrigtsen
2010-11-11 16:22           ` Ted Zlatanov
2010-11-14 17:24             ` Michael Albinus
2010-11-15  0:59               ` Ted Zlatanov [this message]
2010-11-15  4:47                 ` Michael Albinus
2010-11-15 15:14                   ` Ted Zlatanov
2010-11-15 16:03                     ` Michael Albinus
2011-01-24 17:27                       ` auth-source.el rewrite (was: auth-source tokens) Ted Zlatanov
2011-01-24 23:36                         ` auth-source.el rewrite Lars Ingebrigtsen
2011-01-25 16:59                           ` Ted Zlatanov
2011-01-25 21:09                             ` Michael Albinus
2011-01-25 21:42                               ` Ted Zlatanov
2011-01-26  8:32                                 ` Michael Albinus
2011-01-26 17:03                                   ` Ted Zlatanov
2011-01-26 19:35                                     ` Michael Albinus
2011-01-26 20:35                                       ` Ted Zlatanov
2011-01-26 22:15                                         ` Ted Zlatanov
2011-01-27 16:49                                           ` Michael Albinus
2011-01-27 20:20                                             ` Ted Zlatanov
2011-01-29 14:11                                               ` Michael Albinus
2011-01-31  2:49                                                 ` Ted Zlatanov
2011-01-31 14:30                                                   ` Michael Albinus
2011-01-31 17:09                                                     ` Ted Zlatanov
2011-01-27 12:35                                         ` Michael Albinus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tyjjpfkw.fsf@lifelogs.com \
    --to=tzz@lifelogs.com \
    --cc=ding@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).