Skip to content

feat: добавить получение IP-адреса хоста - #6

Merged
Stivo182 merged 1 commit into
mainfrom
feat/ip-host-address
Sep 8, 2026
Merged

feat: добавить получение IP-адреса хоста#6
Stivo182 merged 1 commit into
mainfrom
feat/ip-host-address

Conversation

@Stivo182

@Stivo182 Stivo182 commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Что изменено

  • Добавлен метод URL.IPАдресХоста().
  • Для явно заданных IPv4- и IPv6-хостов метод возвращает неизменяемый IPАдрес.
  • Квадратные скобки IPv6-литерала не входят в результат.
  • Для зарегистрированного имени, IPvFuture, пустого и отсутствующего хоста возвращается Неопределено.
  • ЯвляетсяПетлевым() использует новый метод для обработки IP-хостов.
  • Добавлена документация со ссылками на RFC 3986 и ограничениями применения при защите от SSRF.
  • Добавлены тесты для IPv4, IPv6, IPv4-отображённого IPv6, RegName, IPvFuture и URL без хоста.

Summary by CodeRabbit

  • Новые возможности

    • Добавлен метод получения IP-адреса хоста URL.
    • Для явно указанных IPv4- и IPv6-адресов возвращается объект IP-адреса, включая корректное представление IPv6 без скобок.
    • Для доменных имён, IPvFuture и пустых хостов возвращается Неопределено.
    • DNS-разрешение имён не выполняется; метод не предназначен для самостоятельной защиты от SSRF.
  • Документация

    • Добавлено описание метода, примеры использования, возвращаемые значения и ограничения.

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 716c1ba5-2c67-4b0f-bed1-b2923f03c1cd

📥 Commits

Reviewing files that changed from the base of the PR and between eea6e55 and a0b9ee2.

📒 Files selected for processing (3)
  • docs/URL.md
  • src/Классы/URL.os
  • tests/URL/IPАдресХоста.os

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

URL получил экспортируемый метод IPАдресХоста() для явных IPv4- и IPv6-хостов. Проверка петлевого адреса использует этот метод. Добавлены документация и параметризованные тесты.

Changes

IP-адрес хоста URL

Layer / File(s) Summary
Реализация API адреса хоста
src/Классы/URL.os
Метод IPАдресХоста() возвращает IPАдрес для IPv4- и IPv6-хостов, включая обработку скобок IPv6. Для остальных типов хоста возвращается Неопределено. ЯвляетсяПетлевым() использует новый метод.
Документация и параметризованные проверки
docs/URL.md, tests/URL/IPАдресХоста.os
Документация описывает результат метода, отсутствие DNS-разрешения и ограничения для SSRF. Тесты проверяют IPv4, IPv6, IPv4-mapped IPv6 и хосты без IP-адреса.

Priority: ⬇️ Low — Defer the URL host-IP helper because it is a narrow API addition with documentation and focused IPv4/IPv6 tests, without broader product impact.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to a0b9e

URL теперь извлекает явно заданный IP-адрес хоста и использует эту логику для проверки loopback. Изменение документировано и покрывает IPv4, IPv6 и неподдерживаемые хосты; текущих рисков, блокирующих слияние, не выявлено.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Заголовок точно описывает основное изменение: добавление метода для получения IP-адреса хоста. Формулировка краткая и понятная.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/ip-host-address

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Stivo182
Stivo182 merged commit feca7bc into main Sep 8, 2026
19 checks passed
@Stivo182
Stivo182 deleted the feat/ip-host-address branch September 8, 2026 14:42
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.

1 participant