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.

Sign up for GitHub

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

Jump to bottom

Switching SSH keys to start9 user #1321

Merged
merged 6 commits into from
May 9, 2022
Merged

Switching SSH keys to start9 user #1321

merged 6 commits into from
May 9, 2022

Conversation

k0gen
Copy link
Member

@k0gen k0gen commented Mar 10, 2022

PR for #1306

@ProofOfKeags ProofOfKeags requested a review from dr-bonez March 10, 2022 23:48
@ProofOfKeags ProofOfKeags added startd Issues pertaining to startd itself Community Issues that we would accept PRs for but will not dedicate internal resources to Enhancement New feature or request labels Mar 10, 2022
@dr-bonez
Copy link
Member

looks good, but needs to be tested. Not worth the risk to add it to the patch release, but will likely include it in 0.3.1.

@k0gen
Copy link
Member Author

k0gen commented Mar 11, 2022

Right now I have build and tested it against ENVIRONMENT=dev make on a fresh install

Screen.record.from.2022-03-11.22.55.50.mp4

As you can see .ssh directory ownership is left for root for now so I don't expect any major problems with that.

@dr-bonez dr-bonez added this to the 0.3.1 milestone Mar 23, 2022
@dr-bonez dr-bonez merged commit 10d7a3d into Start9Labs:master May 9, 2022
@k0gen k0gen deleted the patch-1 branch July 12, 2022 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dr-bonez dr-bonez dr-bonez approved these changes

Assignees

@k0gen k0gen

@dr-bonez dr-bonez

Labels
Community Issues that we would accept PRs for but will not dedicate internal resources to Enhancement New feature or request startd Issues pertaining to startd itself
Projects
None yet
Milestone
0.3.1
Development

Successfully merging this pull request may close these issues.
3 participants
@k0gen @dr-bonez @ProofOfKeags