Repository navigation
docs(media): workstations mount the share over 445 as their own user (#523) - #650
Merged
Merged
Conversation
…523) ADR-0051 decides #523. Hicks gets a pass, vlan50 net -> 10.0.40.30:445, described "Allow SMB to smaug", above the CasaBonita block. Workstations log in as a second SMB user, samwise, so bilbo stays the televisions' credential and the two can be revoked separately. A pull from the NAS side (CasaBonita would initiate) and console loading (does not scale) are rejected. Making bilbo read-only is deferred as its own ACL change. build-the-nas.md gains the rule in §0.5's table, a 445 refusal check in §0.6, and §5 "Workstations": create the user, confirm SMB1 is off, create the rule, read its position from morpheus, mount from Hicks, and check that the monitoring host is refused. network.md, security.md, architecture.md, stacks/media/README.md, ADR-0016 and roadmap.md call the pass specified, not created, until those steps are done. The ADR count is now 51. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
3 tasks
Gerrrt
added a commit
that referenced
this pull request
Sep 23, 2026
#650 merged ADR-0051 (Hicks workstations mounting the media share) while this branch carried its own ADR-0051. The filenames differ, so git merged without a conflict and would have left two ADR-0051s. This branch's ADR is now ADR-0052; its references in replace-the-nas-disk.md, hardware.md and its changelog entry follow, and README counts 52 ADRs. Every remaining ADR-0051 reference is #650's. The changelog conflict keeps both entries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #523. This PR decides the issue but does not close it. The issue closes once the steps below have been done at
smaugandmorpheus.Decision (ADR-0051)
vlan50 net → 10.0.40.30:445/tcp, describedAllow SMB to smaug, above Block access to CasaBonita onigc0.50. It is the same kind of rule as the other Hicks passes: one rule per port, and host-scoped. Only445is opened, not NetBIOS on137–139, and the share is reached by address.samwise, for workstations.bilbostays the televisions' credential, unchanged, so the password stored on a TV and the one typed on a laptop can be revoked separately. Makingbilboread-only is deferred, because it means changing the share's ACL.samwise's password can delete or encrypt the library, anderebor/mediais not snapshotted. ADR-0008 treats the library as replaceable.443.445.Documents
build-the-nas.md:445.morpheus, mount from a Hicks workstation and write a test file, confirm the monitoring host is refused, then record the result here.network.md,security.md,architecture.md,stacks/media/README.md, ADR-0016 (a pointer, as for ADR-0050),roadmap.mdandchangelog.mdare updated. All of them call the pass specified, not created.README.mdis now 51.Verification
scripts/check_docs.pyandscripts/lint.shpass. No configuration changed, only documents.After merge
Follow
build-the-nas.md§5 Workstations atsmaugandmorpheus. Then change the four documents from specified to created in one commit, as was done for4533, and close #523.🤖 Generated with Claude Code