Skip to content

fix: обработка null body и некорректного шаблона пути#23

Open
halcyonsdev wants to merge 1 commit intomax-messenger:mainfrom
halcyonsdev:main
Open

fix: обработка null body и некорректного шаблона пути#23
halcyonsdev wants to merge 1 commit intomax-messenger:mainfrom
halcyonsdev:main

Conversation

@halcyonsdev
Copy link

PR исправляет два потенциально опасных крайних случая:

  1. ClientResponse.getBodyAsString() — добавлена проверка на null для поля body, чтобы избежать NullPointerException при ответах без тела (например, HTTP 204 No Content).
  2. MaxQuery, парсинг шаблона пути — добавлена валидация незакрытой фигурной скобки { в шаблоне. Раньше indexOf('}', i) возвращал -1, после чего индекс цикла становился -1 и поведение было некорректным. Теперь
    выбрасывается IllegalArgumentException с понятным сообщением, указывающим на позицию ошибки в шаблоне.

Оба изменения носят защитный характер и не затрагивают существующее поведение при корректных входных данных.

@halcyonsdev halcyonsdev requested a review from a team as a code owner February 25, 2026 09:00
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