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
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.goUnused Fields
PrefixTokenGCloudLocation.ProjectRecoveryCommandImpact
Recommendation
Priority
Low - cleanup/documentation issue