option cannot be used multiple times in __ssh_authorized_keys #179

Closed
opened 2021-11-20 15:20:51 +00:00 by ungleich-gitea · 5 comments

Created by: mtgxx

The manpage says:

option:: an option to set for all created authorized_key entries. Can be specified multiple times.

but it doens't work. The code only handles one option.

Thanks,
Matthias

*Created by: mtgxx* The manpage says: `option:: an option to set for all created authorized_key entries. Can be specified multiple times.` but it doens't work. The code only handles one option. Thanks, Matthias
Author
Owner

Created by: asteven

The code does handle multiple options.
What is the problem you are having?

Err, sorry. You are right.
__ssh_authorized_key does handle multiple options.
__ssh_authorized_keys does not, although it says it does.
This is a bug.

*Created by: asteven* > The code does handle multiple options. > What is the problem you are having? Err, sorry. You are right. __ssh_authorized_key does handle multiple options. __ssh_authorized_keys does not, although it says it does. This is a bug.
Author
Owner

Created by: asteven

The manpage says:

option:: an option to set for all created authorized_key entries. Can be specified multiple times.

but it doens't work. The code only handles one option.

The code does handle multiple options.
What is the problem you are having?

*Created by: asteven* > The manpage says: > > `option:: an option to set for all created authorized_key entries. Can be specified multiple times.` > > but it doens't work. The code only handles one option. > The code does handle multiple options. What is the problem you are having?
Author
Owner

Created by: asteven

Mark Polyakov wrote on 4/7/19 7:54 AM:

I'm also confused about __ssh_authorized_key vs __ssh_authorized_keys -- do we really need 2?

Yes we do.
__ssh_authorized_keys uses __ssh_authorized_key under the hood.

*Created by: asteven* Mark Polyakov wrote on 4/7/19 7:54 AM: > I'm also confused about __ssh_authorized_key vs __ssh_authorized_keys -- do we really need 2? > Yes we do. __ssh_authorized_keys uses __ssh_authorized_key under the hood.
Author
Owner

Created by: markasoftware

I'm also confused about __ssh_authorized_key vs __ssh_authorized_keys -- do we really need 2?

*Created by: markasoftware* I'm also confused about __ssh_authorized_key vs __ssh_authorized_keys -- do we really need 2?
Author
Owner

Created by: telmich

Hey Matthias,

busy @Seoul at the moment - will dig into this next weeks or @asteven might do if he has some time in between - in any case: PR welcome!

Cheers,

Nico

*Created by: telmich* Hey Matthias, busy @Seoul at the moment - will dig into this next weeks or @asteven might do if he has some time in between - in any case: PR welcome! Cheers, Nico
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ungleich-public/cdist#179
No description provided.