Summary: | Не дает вбить IP-адрес сервера | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | Sergey Novikov <sotor> |
Component: | alterator-datetime | Assignee: | Mikhail Efremov <sem> |
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
Severity: | normal | ||
Priority: | P3 | CC: | grenka, manowar, mike, sem |
Version: | unstable | ||
Hardware: | all | ||
OS: | Linux |
Description
Sergey Novikov
2018-07-17 13:18:46 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", если это пригодилось бы в других модулях. (В ответ на комментарий №2) > В конце-концов можно добавить новый тип, какой-нибудь "hostname-or-ip", > если это пригодилось бы в других модулях. Это было бы замечательно :) (В ответ на комментарий №2) > Что требуется сделать? Отключить проверку (тип string) или же можно проверять > валидность в самом бакенде (другой регуляркой). В конце-концов можно добавить Я предпочитаю проверять в самом бакенде. Там все равно проверять, кто знает что и откуда туда приехало. > новый тип, какой-нибудь "hostname-or-ip", если это пригодилось бы в других > модулях. Возможно он уже где-то есть, я точно когда-то хотел такой добавить. Но не уверен, что сделал, мог забить как раз из соображений, что проверки типами - это плохо и лучше делать проверки в самом бакенде. Так ведь одно другому не мешает. Просто проверка в бакенде даёт ошибку типа error ("что-то пошло не так"), ты там можешь не только синтаксис проверять. А валидация при вводе — это вполне самостоятельная вещь. Делаем тогда hostname-or-ip? (В ответ на комментарий №6) > http://git.altlinux.org/tasks/210488/logs/events.1.1.log Проверил на workstation и на kworkstation. IP-адрес успешно добавляется alterator-datetime 4.1-alt1 - Use hostname-or-ip field type for NTP pool address. Спасибо, проверил на своём 801 -- порядок! |