Page MenuHomePhabricator
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
Unknown Object (File)
Sun, Oct 27, 4:09 PM
Unknown Object (File)
Oct 3 2024, 12:24 AM
Unknown Object (File)
Sep 28 2024, 12:23 AM
Unknown Object (File)
Sep 28 2024, 12:23 AM
Unknown Object (File)
Sep 28 2024, 12:23 AM
Unknown Object (File)
Sep 28 2024, 12:23 AM
Unknown Object (File)
Sep 28 2024, 12:19 AM
Unknown Object (File)
Sep 25 2024, 9:55 PM
View All Files
Subscribers
tomek

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
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

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
7
9
23
25
27
This revision now requires changes to proceed.Jul 23 2023, 8:40 AM
• jon updated this revision to Diff 29021.Jul 25 2023, 6:32 AM
• jon marked 6 inline comments as done.

Address feedback

• jon added a comment.Jul 25 2023, 6:32 AM

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.
ashoat added a comment.Jul 28 2023, 8:02 AM

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 ↗(On Diff #29137)
  1. Typo here ("Passwor" should be "Password")
  2. "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 ↗(On Diff #29137)

Please see https://phab.comm.dev/D8660

Revision Contents
Changeset List

PathSize
docs/
1 line
34 lines

Diff 28876

View Options

docs/nix_dev_env.md

Loading...
View Options

docs/nix_services_deployment.md

Loading...