-
Notifications
You must be signed in to change notification settings - Fork 22
Unused configuration fields in LocationConfig #401
Copy link
Copy link
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