Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread Eric Pruitt
On Tue, Jul 26, 2016 at 06:16:15PM +0200, Kajetan Jasztal wrote: > Hash masking [0] password gives you more insight in mistakes you made > while you are typing since you can remember sequence of changing > hashes while you type and correct them on the fly. I've seen that before, and although I th

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread Eric Pruitt
On Tue, Jul 26, 2016 at 06:09:18PM +0200, Jan Christoph Ebersbach wrote: > If dmenu could easily be used as a replacement for pinentry my desktop > would suck a little less. I'm not aware of many other use cases for > entering passwords. Is that possible with your patch? The pinentry programs fo

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread Kajetan Jasztal
2016-07-25 23:22 GMT+02:00 Eric Pruitt : > In general, I find showing the asterisks useful because if I > accidentally press two keys at once or something, I have the opportunity > to correct it rather than typing something then, hitting Enter and > wondering what I did wrong. On a couple of occasi

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread Jan Christoph Ebersbach
If dmenu could easily be used as a replacement for pinentry my desktop would suck a little less. I'm not aware of many other use cases for entering passwords. Is that possible with your patch? Jan Christoph On Mon 25-07-2016 13:21 -0700, Eric Pruitt wrote: > On Mon, Jul 25, 2016 at 11:14:28PM

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread v4hn
Hey everyone, hey hiro, I did not yet look at the patch, but it would be a nice (and imo reasonably small) addition to allow for password entry via dmenu. If the asterisk-approach is too bloated for the maintainer's taste, I'm unsure about this myself, I believe `dmenu -p "Enter Password" -g` wher

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-26 Thread hiro
i think the idea was fine, as long as you provide autocompletion for your password, too.

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 14:40:43 -0700 Eric Pruitt wrote: > I see; I misread "curpos" as "cursor" when I glanced over it the first > time. However, it still looks wrong to me, so I tested it to verify > -- I believe the problem is that the characters aren't guaranteed to > have a width of 8 or even b

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 11:33:18PM +0200, FRIGN wrote: > my suggestion prints as many asterisks as there are runes. try it > out! :D I see; I misread "curpos" as "cursor" when I glanced over it the first time. However, it still looks wrong to me, so I tested it to verify -- I believe the problem i

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 14:22:16 -0700 Eric Pruitt wrote: Hey Eric, > Personally, I like having the asterisks reflect the actual number of > runes typed and think the cosmetics are worth the extra lines. Even if > you're typing Latin characters, I think only getting feedback every 8 > key presses ki

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 11:10:26PM +0200, FRIGN wrote: > I looked more closely at the patch and I've found potential to simplify > it. Why not go with a [...] Personally, I like having the asterisks reflect the actual number of runes typed and think the cosmetics are worth the extra lines. Even if

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 11:39:55 -0700 Eric Pruitt wrote: > The attached patch adds a "-g" flag to dmenu so it can be used for > password entry. Typed text is replaced with asterisks. I looked more closely at the patch and I've found potential to simplify it. Why not go with a static char as

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Martin Kühne
Use more key files. That counts especially for gpg, ssh as well as luks. This thread sounds like an attempt at slowly replacing whatever you did with your terminal emulator before with dmenu. Does one line of text suck less than several? cheers! mar77i

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 10:21:09PM +0200, FRIGN wrote: > On Mon, 25 Jul 2016 13:07:23 -0700 > Eric Pruitt wrote: > > > Again, no one is saying passwords should be sent via the command line. > > Look at the patch. It uses stdout just like vanialla dmenu. > > Ah I see, thanks for clearing up that pa

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 11:14:28PM -1100, mikan wrote: > I was working on a similar thing but decided that password entry doesn't make > sense in dmenu context (since it's a menu!) and made a separate tool instead > and glue pinentry code to it. I don't agree with this sentiment -- dmenu happily s

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 13:07:23 -0700 Eric Pruitt wrote: > Again, no one is saying passwords should be sent via the command line. > Look at the patch. It uses stdout just like vanialla dmenu. Ah I see, thanks for clearing up that part. Now, what do the other people think about it? Cheers FRIGN -

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread mikan
I was working on a similar thing but decided that password entry doesn't make sense in dmenu context (since it's a menu!) and made a separate tool instead and glue pinentry code to it. You can look at it at http://gitgud.io/zavok/spine if you want. On July 25, 2016 7:39:55 AM GMT-11:00, Eric Pru

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 01:10:01PM -0700, Eric Pruitt wrote: > On Mon, Jul 25, 2016 at 10:06:47PM +0200, FRIGN wrote: > > I really would like to see if "example" actually exists. Does it > > really make sense to do that and is it even safe? > > Off the top of my head. both GPG and luks accept passw

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 10:06:47PM +0200, FRIGN wrote: > I really would like to see if "example" actually exists. Does it > really make sense to do that and is it even safe? Off the top of my head. both GPG and luks accept passwords and keyfiles via arbitrary files which can be regular files or pi

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 10:03:34PM +0200, FRIGN wrote: > So you pass passwords on the command line? Again, an example is really > overdue. I have no clue what you're talking about. What makes you think I'm sending passwords over the command line? Dmenu sends the selection to stdout, not argv, and

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 12:53:04 -0700 Eric Pruitt wrote: > And to clear the terminology: With "command line", I mean inside the actual arg-array, like $ example "supersecretpassword" If you actually mean a program that gets the password piped like $ dmenu ... | example I really wo

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 12:53:04 -0700 Eric Pruitt wrote: Hey Eric, > There's an example in the description: entering passwords. I don't > think any thing else needs to be said, but I will humor you: So you pass passwords on the command line? Again, an example is really overdue. > > For me, this d

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread Eric Pruitt
On Mon, Jul 25, 2016 at 09:45:16PM +0200, FRIGN wrote: > On Mon, 25 Jul 2016 11:39:55 -0700 > > The attached patch adds a "-g" flag to dmenu so it can be used for > > password entry. Typed text is replaced with asterisks. > > Why would you need that? Can you give an example? There's an example in

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

2016-07-25 Thread FRIGN
On Mon, 25 Jul 2016 11:39:55 -0700 Eric Pruitt wrote: Hey Eric, > The attached patch adds a "-g" flag to dmenu so it can be used for > password entry. Typed text is replaced with asterisks. Why would you need that? Can you give an example? For me, this doesn't make any bloody sense and gives a