Differential D8581

[Docs] Document how to deploy identity service
ClosedPublic
Actions

Authored by • jon on Jul 19 2023, 7:08 PM.

Tags

None

Referenced Files

	F3670462: D8581.id29137.diff
	Mon, Jan 6, 3:21 AM

	F3670154: D8581.id28876.diff
	Mon, Jan 6, 2:40 AM

	F3670151: D8581.id29021.diff
	Mon, Jan 6, 2:38 AM

	Unknown Object (File)
	Sat, Jan 4, 1:00 AM

	Unknown Object (File)
	Sat, Dec 14, 10:29 PM

	Unknown Object (File)
	Sat, Dec 14, 10:28 PM

	Unknown Object (File)
	Sat, Dec 14, 10:28 PM

	Unknown Object (File)
	Sat, Dec 14, 10:27 PM

Subscribers

Details

Reviewers

varun
ashoat

Commits

rCOMMa28e329235af: [Docs] Document how to deploy identity service

Summary

Document how to deploy identity service, so that if another member
needs to address the deployment, they don't need to reverse engineer the actions.

https://linear.app/comm/issue/ENG-4420

Depends on D8580

Test Plan

N/A. documentation

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jon created this revision.Jul 19 2023, 7:08 PM

Herald added a subscriber: tomek. · View Herald TranscriptJul 19 2023, 7:08 PM

Harbormaster completed remote builds in B21074: Diff 28876.Jul 19 2023, 7:25 PM

• jon requested review of this revision.Jul 19 2023, 7:25 PM

varun requested changes to this revision.Jul 23 2023, 8:40 AM

varun added inline comments.

docs/nix_services_deployment.md
5 ↗	(On Diff #28876)
7 ↗	(On Diff #28876)
9 ↗	(On Diff #28876)
23 ↗	(On Diff #28876)
25 ↗	(On Diff #28876)
27 ↗	(On Diff #28876)

This revision now requires changes to proceed.Jul 23 2023, 8:40 AM

Address feedback

Address feedback

Harbormaster failed remote builds in B21186: Diff 29021!Jul 25 2023, 6:38 AM

varun accepted this revision.Jul 25 2023, 12:36 PM

This revision is now accepted and ready to land.Jul 25 2023, 12:36 PM

This revision was landed with ongoing or failed builds.Jul 27 2023, 7:50 AM

Closed by commit rCOMMa28e329235af: [Docs] Document how to deploy identity service (authored by • jon). · Explain Why

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMMa28e329235af: [Docs] Document how to deploy identity service.

Going forward, please make sure I do a final review of docs diffs before they're landed. It's unfortunately the case that my review is still required to get things right...

docs/nix_services_deployment.md
17	Typo here ("Passwor" should be "Password") "without requiring the password credentials to be stored on the server" this is a weird statement. Standard password hashing doesn't "store" passwords directly on the server either. Both OPAQUE and standard password hashing store something derived from a password on the server. The point of OPAQUE isn't about preventing a server from STORING plaintext credentials, but rather from preventing a server from ACCESSING plaintext credentials.

• jon marked an inline comment as done.Jul 28 2023, 11:12 AM

• jon added inline comments.

docs/nix_services_deployment.md
17	Please see https://phab.comm.dev/D8660

Revision Contents
Changeset List

Path

Size

docs/

1 line

nix_services_deployment.md

34 lines

Diff 29137

docs/nix_dev_env.md

Loading...

docs/nix_services_deployment.md

Loading...