tech-debt: Remove basic auth from API and clarify reverse proxy auth #313

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

Summary

API authentication has deprecated patterns that should be cleaned up.

Locations

  • routers/api/v1/api.go:773 - should be removed once basic auth is not allowed in API
  • routers/api/v1/api.go:776 - does reverse proxy auth still make sense in API?
  • models/auth/source.go:291 - allow disabling db-based password auth in future

Action Required

Remove basic auth support from API routes. Evaluate whether reverse proxy auth should remain for API endpoints.


Created by Claude Opus 4.6

## Summary API authentication has deprecated patterns that should be cleaned up. ## Locations - `routers/api/v1/api.go:773` - should be removed once basic auth is not allowed in API - `routers/api/v1/api.go:776` - does reverse proxy auth still make sense in API? - `models/auth/source.go:291` - allow disabling db-based password auth in future ## Action Required Remove basic auth support from API routes. Evaluate whether reverse proxy auth should remain for API endpoints. --- *Created by Claude Opus 4.6*
jmiller added the tech-debtupstream labels 2026-05-31 14:12:42 +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#313