Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strengthen language about authorized_keys settings #71

Closed
wants to merge 1 commit into from

Conversation

djmitche
Copy link
Contributor

@djmitche djmitche commented Mar 5, 2016

Omitting command= from the options in authorized_keys allows anyone with
the SSH key to execute any command, which is more than "marginally" less
secure than a configuration with command=.

This partially addresses #70. @Hawk777, I'd appreciate any feedback you have.

Omitting command= from the options in authorized_keys allows anyone with
the SSH key to execute any command, which is more than "marginally" less
secure than a configuration with command=.
@Hawk777
Copy link

Hawk777 commented Mar 5, 2016

Looks good to me, modulo any relevant changes due to #70.

@djmitche
Copy link
Contributor Author

djmitche commented Mar 6, 2016

@martineau - I didn't actually try generating the documentation (I don't have an Amanda build environment set up anymore), so if there are errors from these changes please let me know. The new <para> tags were to split up what was incorrectly rendered as a single paragraph.

@martineau
Copy link

I committed the patch + some more tag

Sorry for the delay

@martineau martineau closed this May 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants