Paths

Table of Contentst

Differential D11700

Correct private key permissions before extracting public key in bin/almanac register
ClosedPublic
Actions

Authored by epriestley on Feb 5 2015, 10:05 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 4, 6:39 PM

	Unknown Object (File)
	Thu, May 2, 7:15 AM

	Unknown Object (File)
	Tue, Apr 30, 11:19 PM

	Unknown Object (File)
	Sat, Apr 27, 10:33 AM

	Unknown Object (File)
	Wed, Apr 17, 2:56 PM

	Unknown Object (File)
	Thu, Apr 11, 7:09 AM

	Unknown Object (File)
	Tue, Apr 9, 7:43 AM

	Unknown Object (File)
	Tue, Apr 9, 7:43 AM

Subscribers

Details

Reviewers

Commits

Restricted Diffusion Commit
rP57f1ab705e28: Correct private key permissions before extracting public key in bin/almanac…

Summary

ssh-keygen declines to run on a too-public key. Write the correctly-restricted key a little earlier in the workflow.

Test Plan

epriestley@orbital ~/dev/phabricator $ chmod 644 ~/dev/core/conf/keys/daemon.key 
epriestley@orbital ~/dev/phabricator $ ./bin/almanac register --private-key ~/dev/core/conf/keys/daemon.key --identify-as local.phacility.net --device daemon.phacility.net --force --allow-key-reuse
Installing public key...
Installing private key...
Installing device ID...
 HOST REGISTERED  This host has been registered as "local.phacility.net" and a trusted keypair has been installed.
epriestley@orbital ~/dev/phabricator $

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 28154.Feb 5 2015, 10:05 PM

epriestley retitled this revision from to Correct private key permissions before extracting public key in bin/almanac register.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptFeb 5 2015, 10:05 PM

btrahan accepted this revision.Feb 5 2015, 10:07 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Feb 5 2015, 10:07 PM

Closed by commit rP57f1ab705e28: Correct private key permissions before extracting public key in bin/almanac… (authored by epriestley, committed by epriestley). · Explain WhyFeb 5 2015, 10:09 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

almanac/

management/

AlmanacManagementRegisterWorkflow.php

16 lines

Diff 28156

src/applications/almanac/management/AlmanacManagementRegisterWorkflow.php

Loading...