Skip to content

[Glense] Security hardening & inter-service auth improvements#28

Merged
bogdans55 merged 1 commit intomainfrom
bogdans55/security-improvements
Mar 31, 2026
Merged

[Glense] Security hardening & inter-service auth improvements#28
bogdans55 merged 1 commit intomainfrom
bogdans55/security-improvements

Conversation

@bogdans55
Copy link
Copy Markdown
Collaborator

Summary

  • Move all hardcoded secrets (JWT keys, DB passwords, RabbitMQ credentials) from docker-compose.yml to .env file with
    .env.example template
  • Add inter-service API key authentication (INTERNAL_API_KEY):
    • gRPC server interceptor on Account Service validates x-internal-api-key header
    • gRPC client interceptor on Video Catalogue attaches the key automatically
    • Donation Service HTTP client sends the key when calling Account Service
  • Restrict CORS on all backend services from AllowAnyOrigin to config-driven whitelist
  • Fix GetCurrentUserId() crash in Account Service (Guid.Parse → TryParse)
  • Remove hardcoded JWT fallback in Chat Service
  • Clean up README: fix architecture diagram alignment, remove incorrect database schema section and SVG
  • Update DEV_QUICKSTART.md to reflect new .env-based setup

@bogdans55 bogdans55 requested a review from Brankonymous March 31, 2026 15:08
@Brankonymous Brankonymous changed the title Security hardening & inter-service auth improvements [Glense] Security hardening & inter-service auth improvements Mar 31, 2026
@Brankonymous Brankonymous added this to the sprint-3 milestone Mar 31, 2026
@Brankonymous Brankonymous moved this to Code Review in Glense Mar 31, 2026
@bogdans55 bogdans55 merged commit 325d6cd into main Mar 31, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Code Review to Done in Glense Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants