Skip to content

Unused configuration fields in LocationConfig #401

@wilke

Description

@wilke

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

  1. Either implement the intended functionality for these fields
  2. Or remove them from the struct and document why they're not needed

Priority

Low - cleanup/documentation issue

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions