-
Notifications
You must be signed in to change notification settings - Fork 22
Open
Description
Summary
Several fields in LocationConfig are parsed from YAML but never used in the codebase, creating confusion about what configuration options actually work.
Location
shock-server/conf/conf.go
Unused Fields
| Field | Line | Notes |
|---|---|---|
Prefix |
32 | Parsed but never referenced |
Token |
30 | Redundant with AuthKey/SecretKey |
GCloudLocation.Project |
62 | Only Bucket/SecretKey used |
RecoveryCommand |
- | TSM handled by external scripts |
Impact
- Users may configure these fields expecting behavior changes
- Documentation may reference features that don't work
- Code maintenance burden for unused fields
Recommendation
- Either implement the intended functionality for these fields
- Or remove them from the struct and document why they're not needed
Priority
Low - cleanup/documentation issue
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels