Bug 35160 - Не дает вбить IP-адрес сервера
Summary: Не дает вбить IP-адрес сервера
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-datetime (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Mikhail Efremov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-17 13:18 MSK by Sergey Novikov
Modified: 2018-07-19 14:55 MSK (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Novikov 2018-07-17 13:18:46 MSK
Не дает в качестве адреса NTP сервера указать IP-адрес
Comment 1 Michael Shigorin 2018-07-18 09:47:22 MSK
Довольно срочное -- Паш, если Миша сейчас не сможет, просьба глянуть.
Comment 2 manowar@altlinux.org 2018-07-18 12:22:51 MSK
(В ответ на комментарий №0)
> Не дает в качестве адреса NTP сервера указать IP-адрес

Сие не удивительно: там тип указан "hostname" вот с такой регуляркой: http://git.altlinux.org/gears/a/alterator.git?p=alterator.git;a=blob;f=alterator/type/hostname.scm;h=4af353a8137655f58652da01c17d49717b70e927;hb=70b252b7c67916bef25bcfd28099322df5a19e02 .

Что требуется сделать? Отключить проверку (тип string) или же можно проверять валидность в самом бакенде (другой регуляркой). В конце-концов можно добавить новый тип, какой-нибудь "hostname-or-ip", если это пригодилось бы в других модулях.
Comment 3 Michael Shigorin 2018-07-18 12:24:40 MSK
(В ответ на комментарий №2)
> В конце-концов можно добавить новый тип, какой-нибудь "hostname-or-ip",
> если это пригодилось бы в других модулях.
Это было бы замечательно :)
Comment 4 Mikhail Efremov 2018-07-18 12:30:18 MSK
(В ответ на комментарий №2)
> Что требуется сделать? Отключить проверку (тип string) или же можно проверять
> валидность в самом бакенде (другой регуляркой). В конце-концов можно добавить

Я предпочитаю проверять в самом бакенде. Там все равно проверять, кто знает что и откуда туда приехало.

> новый тип, какой-нибудь "hostname-or-ip", если это пригодилось бы в других
> модулях.

Возможно он уже где-то есть, я точно когда-то хотел такой добавить. Но не уверен, что сделал, мог забить как раз из соображений, что проверки типами - это плохо и лучше делать проверки в самом бакенде.
Comment 5 manowar@altlinux.org 2018-07-18 12:32:44 MSK
Так ведь одно другому не мешает. Просто проверка в бакенде даёт ошибку типа error ("что-то пошло не так"), ты там можешь не только синтаксис проверять. А валидация при вводе — это вполне самостоятельная вещь. Делаем тогда hostname-or-ip?
Comment 6 manowar@altlinux.org 2018-07-18 16:33:56 MSK
http://git.altlinux.org/tasks/210488/logs/events.1.1.log
Comment 7 Sergey Novikov 2018-07-18 16:48:03 MSK
(В ответ на комментарий №6)
> http://git.altlinux.org/tasks/210488/logs/events.1.1.log

Проверил на workstation и на kworkstation. IP-адрес успешно добавляется
Comment 8 manowar@altlinux.org 2018-07-19 01:17:45 MSK
alterator-datetime 4.1-alt1
- Use hostname-or-ip field type for NTP pool address.
Comment 9 Michael Shigorin 2018-07-19 14:55:31 MSK
Спасибо, проверил на своём 801 -- порядок!