tech-debt: Fix ACME/TLS setup path issues and install flow #321

Open
opened 2026-05-31 14:22:06 +00:00 by jmiller · 0 comments
Owner

Summary

ACME certificate management and installation have known issues.

Locations

  • cmd/web_acme.go:46 - placeholders until settings options with warnings are added
  • cmd/web_acme.go:68 - path uses AppWorkPath incorrectly, writes into AppWorkPath/https
  • cmd/web.go:243 - should use a proper context
  • routers/install/install.go:415 - oauth2 JWT_SECRET generated regardless of oauth2 enabled state

Action Required

Fix ACME data path to use correct directory. Add proper settings UI for ACME. Fix oauth2 secret generation logic.


Created by Claude Opus 4.6

## Summary ACME certificate management and installation have known issues. ## Locations - `cmd/web_acme.go:46` - placeholders until settings options with warnings are added - `cmd/web_acme.go:68` - path uses AppWorkPath incorrectly, writes into AppWorkPath/https - `cmd/web.go:243` - should use a proper context - `routers/install/install.go:415` - oauth2 JWT_SECRET generated regardless of oauth2 enabled state ## Action Required Fix ACME data path to use correct directory. Add proper settings UI for ACME. Fix oauth2 secret generation logic. --- *Created by Claude Opus 4.6*
jmiller added the tech-debtupstream labels 2026-05-31 14:22:06 +00:00
Sign in to join this conversation.
No labels tech-debt upstream
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGitea#321