Skip to content

Conversation

@gavlyukovskiy
Copy link
Collaborator

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes OpenAPI configuration behind a reverse proxy and updates Docker deployment settings. The changes make the OpenAPI suffix configurable and add restart policies to all Docker services.

Key changes:

  • Made OpenAPI suffix configurable via properties/environment variables instead of hardcoding domain checks
  • Added restart: on-failure policy to all Docker services for improved reliability
  • Changed frontend nginx to listen on port 3000 instead of port 80

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
frontend/nginx.conf Changed nginx listen port from 80 to 3000
deployment/docker-compose.yaml Added restart policies, service dependencies, increased JVM cache, and OpenAPI suffix environment variable
deployment/README.md Added operations documentation for SSH access, Docker stats, and log checking
backend/src/main/resources/application.properties Added OpenAPI suffix configuration and Swagger UI URL properties
backend/src/main/java/net/hackyourfuture/coursehub/SwaggerBehindReverseProxyConfig.java Replaced hardcoded domain check with configurable OpenAPI suffix
backend/src/main/java/net/hackyourfuture/coursehub/CourseHubApplication.java Added ConfigurationPropertiesScan import (unused in shown diff)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

…plication.java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@gavlyukovskiy gavlyukovskiy merged commit 60852ab into main Oct 19, 2025
2 checks passed
@Breus Breus deleted the fixes branch October 21, 2025 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants