Modify

#814 closed enhancement (duplicate)

Can't use /oper to change passwords of existing accounts.

Reported by: nbowler@… Owned by:
Priority: normal Milestone:
Component: BitlBee Version: 3.0.3
Keywords: Cc:
IRC client+version: Client-independent Operating System: Linux
OS version/distro:

Description

When adding a new account, one can omit the password and use /oper to set it, which typically allows the password to be entered without characters being printed to the screen and without the password being written to a log file.

However, the same functionality seems to be missing from the "account N set password" command. With this command, if you omit the actual password it just prints a (rather useless, imo) eight asterisks instead of allowing you to use /oper to change it.

It would be nice for this functionality to exist for password changes, or (if it actually does exist and I just can't figure out how to use it), it should be mentioned in "help set password".

Attachments (1)

0001-Oper-noload-force-identify-fix.patch (2.2 KB) - added by Artem Savkov <artem.savkov@…> at 2012-03-18T09:58:54Z.

Download all attachments as: .zip

Change History (11)

comment:1 Changed at 2011-07-24T12:09:31Z by wilmer

"set $SETTING" shows its current value. Hardcoding completely different behaviour for the password field seems like a bad idea.

Maybe some "account X resetpassword" command would do it. I'll keep it in mind.

comment:2 Changed at 2012-02-25T16:22:31Z by anonymous

Anything like that possible, atm?

regards

comment:3 Changed at 2012-03-12T23:51:10Z by wilmer

Merging with #815 for /oper for identify -noload/-force.

No clue when I'll get to this. Patches are welcome. We just need a few extra flags defined in irc.h.

Changed at 2012-03-18T09:58:54Z by Artem Savkov <artem.savkov@…>

comment:4 Changed at 2012-03-18T09:59:08Z by Artem Savkov <artem.savkov@…>

Attached my take on -noload/-force thingie. So whats the final decision on "set password"? Should it be a new command?

comment:5 Changed at 2012-03-18T10:48:34Z by Wilmer van der Gaast <wilmer@…>

I wonder if "set -del" could be used for that somehow.

comment:6 in reply to:  5 Changed at 2012-03-18T12:39:42Z by Artem Savkov <artem.savkov@…>

Replying to Wilmer van der Gaast <wilmer@…>:

I wonder if "set -del" could be used for that somehow.

That sounds easier than "set password", but I'm not sure if using -del like this would be clear to user. btw adding SET_HIDDEN flag to "password" variable may make things a bit easier.

comment:7 Changed at 2012-03-18T12:41:21Z by Wilmer van der Gaast <wilmer@…>

Yeah, but that'd hide the setting of the setting entirely which is definitely confusing. I'd rather not.

Right now "account add" has special behaviour for NULL passwords. This should be reusable for set -del password, at least for IM account passwds. Feel free to leave this BTW, this gets a bit hairy so I might as well take a look myself.

comment:8 Changed at 2012-03-19T21:01:00Z by wilmer

Submitted. I did need to make some more changed to make it work..

flags |= 0x010 and then if( flags & 0x100 ) { ... } won't do much. :-)

comment:9 Changed at 2014-02-04T06:10:01Z by dx

See ticket #1117

comment:10 Changed at 2014-07-07T11:50:47Z by dx

Resolution: duplicate
Status: newclosed

With ticket #1117 being fixed, and the other patch in this ticket being merged too, I guess I can close this.

Modify Ticket

Action
as closed The ticket will remain with no owner.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.