Skip to content

fix(compose): Fix port forwarding to localhost only#21

Merged
Fl1riX merged 1 commit into
mainfrom
fix/port-availability
May 25, 2026
Merged

fix(compose): Fix port forwarding to localhost only#21
Fl1riX merged 1 commit into
mainfrom
fix/port-availability

Conversation

@Fl1riX
Copy link
Copy Markdown
Owner

@Fl1riX Fl1riX commented May 25, 2026

Summary by Sourcery

Build:

  • Обновить сопоставление портов в docker-compose, чтобы привязать порт службы API 8000 к 127.0.0.1 вместо всех интерфейсов.
Original summary in English

Summary by Sourcery

Build:

  • Update docker-compose port mapping to bind API service port 8000 to 127.0.0.1 instead of all interfaces.

@sourcery-ai
Copy link
Copy Markdown
Contributor

sourcery-ai Bot commented May 25, 2026

Инструкция для ревьюера (свернута для маленьких PR)

Инструкция для ревьюера

Этот PR ужесточает правила экспонирования портов в Docker Compose, привязывая порт 8000 сервиса API только к localhost вместо всех сетевых интерфейсов.

Изменения на уровне файлов

Изменение Подробности Файлы
Ограничить экспонирование порта сервиса API только localhost в Docker Compose.
  • Изменить настройку ports у сервиса api, чтобы добавить явную привязку хоста 127.0.0.1 для порта 8000
  • Убедиться, что порт 8000 контейнера по-прежнему доступен через localhost:8000, но больше не экспонируется на всех сетевых интерфейсах
docker-compose.yml

Подсказки и команды

Взаимодействие с Sourcery

  • Запустить новое ревью: Оставьте комментарий @sourcery-ai review в pull request.
  • Продолжить обсуждения: Отвечайте напрямую на комментарии ревью от Sourcery.
  • Создать задачу GitHub из комментария ревью: Попросите Sourcery создать
    issue из комментария ревью, ответив на него. Вы также можете ответить на
    комментарий ревью с @sourcery-ai issue, чтобы создать из него issue.
  • Сгенерировать заголовок pull request: Напишите @sourcery-ai в любом месте
    заголовка pull request, чтобы сгенерировать заголовок в любой момент. Также можно
    оставить комментарий @sourcery-ai title в pull request, чтобы (пере)сгенерировать
    заголовок в любой момент.
  • Сгенерировать краткое резюме pull request: Напишите @sourcery-ai summary
    в любом месте тела pull request, чтобы сгенерировать резюме PR в любой момент
    прямо там, где вы хотите. Также можно оставить комментарий
    @sourcery-ai summary в pull request, чтобы (пере)сгенерировать резюме в любой момент.
  • Сгенерировать инструкцию для ревьюера: Оставьте комментарий @sourcery-ai guide
    в pull request, чтобы (пере)сгенерировать инструкцию для ревьюера в любой момент.
  • Разрешить все комментарии Sourcery: Оставьте комментарий @sourcery-ai resolve
    в pull request, чтобы разрешить все комментарии Sourcery. Полезно, если вы уже
    учли все комментарии и больше не хотите их видеть.
  • Отклонить все ревью Sourcery: Оставьте комментарий @sourcery-ai dismiss
    в pull request, чтобы отклонить все существующие ревью Sourcery. Особенно полезно,
    если вы хотите начать с чистого листа с новым ревью — не забудьте оставить
    @sourcery-ai review, чтобы запустить новое ревью!

Настройка вашего опыта

Зайдите в свою панель управления, чтобы:

  • Включать или отключать функции ревью, такие как сгенерированное Sourcery
    резюме pull request, инструкцию для ревьюера и другие.
  • Изменить язык ревью.
  • Добавлять, удалять или редактировать пользовательские инструкции для ревью.
  • Настраивать другие параметры ревью.

Получение помощи

Original review guide in English
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This PR tightens Docker Compose port exposure by binding the API service’s port 8000 to localhost only instead of all interfaces.

File-Level Changes

Change Details Files
Restrict API service port exposure to localhost in Docker Compose.
  • Modify the api service ports mapping to include an explicit 127.0.0.1 host binding for port 8000
  • Ensure the container’s port 8000 is still accessible via localhost:8000 while no longer exposed on all network interfaces
docker-compose.yml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Привет — я просмотрел(а) твои изменения, и они выглядят отлично!


Sourcery бесплатен для open source — если тебе нравятся наши обзоры, пожалуйста, поделись ими ✨
Помоги мне стать полезнее! Пожалуйста, нажимай 👍 или 👎 под каждым комментарием, и я воспользуюсь твоим отзывом, чтобы улучшить обзоры.
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Fl1riX Fl1riX merged commit 37967b1 into main May 25, 2026
4 checks passed
@Fl1riX Fl1riX deleted the fix/port-availability branch May 25, 2026 14:27
@coveralls
Copy link
Copy Markdown

coveralls commented May 25, 2026

Coverage Report for CI Build 26405403290

Warning

No base build found for commit 4c556aa on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 28.248%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1147
Covered Lines: 324
Line Coverage: 28.25%
Coverage Strength: 0.56 hits per line

💛 - Coveralls

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