такой как дьячук и выкинет)

Хотя, мне с другой стороны непонятно, как вы собрались писать проактивного бота на веб хуках. Он же только работает по модели запрос-ответ, а сам ничего инициировать не может.

А как кроме веб хуков писать. Это один из двух вариантов взаимодействия с ботом. Второй как показала практика не очень.

прост в конторе не был, где дрочат за тесты. Функционал не покрыт тестами нормально, ПР не примут. А долго будешь мусолить вылетишь)

Норм буду знать

Да это критерий отбора даже для джунов. Вон на рубиджобс полистать. Везде тесты

Ну я на второй вариант и намекал. Бот в виде постоянно работающего демона. Тянет с телеграма апдейты, парсит их, отсылает сообщения и т.д.

Через get updates это же печаль

типа не феншуй

Всю жизнь так боты работали - для IRC, ICQ, XMPP, Skype. А тут вам чето не феншуй :\

Писал я. Видел как на хероку сервере он каждую секунду фигарит одну и ту же лог функцию

ты про getUpdates ?

Ну

Там можно устанавливать таймаут для отключения. Можно 30 секунд а не 1

А ну тогда норм. Только если таймаут 30 секунд то бот заторможенный становится не?

Нет, это будет аналог long polling механизма. Как только получаешь апдейт, тут же обрабатываешь его и идешь дальше висеть ждать запроса.

Если таймаут наступил, то тут же опять делаешь запрос на getUpdates и висишь еще 30 секунд.

А типо фаза сна и фаза бодроствования

Ну да. А в это время можешь в отдельном треде по таймеру рассылать спам по контактам :) Получается полноценный живой бот а не тупо реагирующий на входящий веб-хук.

у меня бот на руби на веб хуках

делать лонг поллинг в 2016 - это моветон

Привет. попал в тупик, есть статья, есть коменнатрии к ней, хочу сделать так чтобы коментарии мог удалять только хозяин коментария или статьи, сделал before_action :correct_user перед destroy экшном в CommentsControler выглядет так: def correct_user if current_user == nil flash[:danger] = t(:dont_allow_delet_comment) redirect_to @article end if !current_user.articles.include?(@article) || !current_user.comments.include?(@comment) flash[:danger] = t(:dont_allow_delete_comment) redirect_to @article end end

но не работает как я ожидаю гости немогут удалять как и полагаеться хозяин статьи может удаолять как и пологаеться но вот в чужой статье, свой комментарий ползователь удалить не может ломаю голову почему=( уверен ответ простой... =(

if comment.author == current_user else ?

в случае чужой статьи if !current_user.articles.include?(@article) равно true

вторую половину оно не смотрит

1. Давайте назовем before_action семантически корректно. Суть экшна в чем? Проверить, может ли пользователь удалить комментарий. authorize_comment_destroy, или просто authorize_destroy, учитывая, что мы находимся в контроллере комментариев и понятно, что речь идет о комментарии. 2. У нас несколько разных условий, но одно и то же следствие - ошибка во flash и редирект. Следствие можно вынести в отдельный метод. def authorize_destroy return authorize_failed unless current_user.present? return authorize_failed if @article.owner != current_user return authorize_failed if @comment.owner != current_user true end def authorize_failed flash[:danger] = t(:dont_allow_delete_comment) redirect_to @article end

плюсик Виталию, а то куча if’ов выглядит малочитабельно

Написал как посоветовали, спасибо, но юзер в чужой статье всеровно не может удалять свои коментарии=(

return authorize_failed if @article.owner != current_user return authorize_failed if @comment.owner != current_user верю что можно написать 1 условие к current_user

Написал как посоветовали, спасибо, но юзер в чужой статье всеровно не может удалять свои коментарии=(

потому что у тебя обрывается на втором условии return authorize_failed if @article.owner != current_user

дальше оно не смотрит

понял) с || тоже не рабьотает

ка быть то

=(

А почему нельзя простую проверку делать current_user == comment.author ?

Тогда не важно чья статья

ну у коммент может быть вообще без автора

Поиск его коммента по куки или он просто оставил и все коммент навсегда ничейный?

т.е. если нашелся по кукисам, то может удалить, если нет, то все прощай

навсегда

А его нужно удалять кому теперь, админу?

хозяину статьи

вроже рабюоатет всем спасибо

А связи как настроены между статьей и комментами

статьи имеют много коментов

Кстати, можно было бы вызывать метод correct_user, если только ты не автор статьи, иначе спокойно destroy comment

def reject_user flash[:danger] = t(:dont_allow_delet_comment) redirect_to @article end def check_user_permissions reject_user if current_user.nil? if !current_user.comments.include?(@comment) && !current_user.articles.include?(@article) reject_user end end

что насчет такого?)

я бы вообще в модель Comment впилил before_destroy :authorize_failed def authorize_failed return if owner == current_user || article.owner == current_user errors.add(:owner, 'Whoops') throw :abort end

тонкие контроллеры, все дела

в любом случае надо проверять что destroy возвратил

Вообще надо типа Cancacan или Pundit использовать в этом случае

ох уж эти рельсовики, на каждый чих гем тащить

чем оно лучше callback'a здесь?

проще написать свой небольшой аналог библиотеки работы с правами

before_destroy :authorize_failed def authorize_failed return if owner == current_user || article.owner == current_user errors.add(:owner, 'Whoops') throw :abort end это в модели из коробки вообще работать не будет, там current_user нет

гем лучше тащить потому что это задел на будущее

потом проще расширять

да и pundit легкий по сути

а я бы для взял Pundit илиcancancan и сделал нормальные полиси

before_destroy :authorize_failed def authorize_failed return if owner == current_user || article.owner == current_user errors.add(:owner, 'Whoops') throw :abort end это в модели из коробки вообще работать не будет, там current_user нет

ты так говоришь как будто его нельзя пробросить в модель

собственно у сеня в бложике так и сделал. если у тебя нет прав то в скоупе ты этого комента даже не увидишь

ты так говоришь как будто его нельзя пробросить в модель

"из коробки" же он говорит

ERROR: S client not available

точнее в бложике скоупов нет, но если ты не автор коментов/поста/модератор - фиг тебе а не правка/удаление

c && правильно)

pundit не стал тащить потомучто это самая сложная проверка в прилодении

pundit не стал тащить потомучто это самая сложная проверка в прилодении

ну от приложения зависит, но писать свои костыли когда тебе авторизация и аутентификация всё равно нужна - так себе идея. опять же, у меня допустим ни в моделях и в контроллерах логики почти нет сейчас, у меня в полиси Pundit'а прописаны скоупы, и юзер физически не сможет получить модель по id, которой в скоупе нет

я делал так в rails in action, удобно, но я какбы это приложение чисто для себя делаю, да и моделей кроме коментов артиклй и юзера больше не будет

я делал так в rails in action, удобно, но я какбы это приложение чисто для себя делаю, да и моделей кроме коментов артиклй и юзера больше не будет

можешь посмотреть как у меня сделано https://github.com/foxmuldercp/blogapp

можешь посмотреть как у меня сделано https://github.com/foxmuldercp/blogapp

а что означает цп в конце ника?

был такой фан клуб аниме, кристал повер, шоб в ирц не путать, доцепил постфик

этому нику 15+ лет

во времена видеокассет и диалапа

33 в декабре

т.е. тебе за 40?

нет. 33 же. админского стажа с 96, официально с 2003

админ. программить в 12м начал, как раз студия вышла

понял

админ. программить в 12м начал, как раз студия вышла

это какой год поточнее?

ну 2012

ааа

подумал в 12 классе

и что за студия? а то подумал на Visual Studio C++ 6

она примерно тогда и вышла

в 2001 +- 2 года

предлагаю в флеймилку свалить, а то мне тут это, неудобно распальцованными граблями печатать :)

флеймилка?

https://telegram.me/rubyandrailsflame

господи

ЕЩЁ БОЛЬШЕ ЧАТОВ

это неша флеймилка

у меня бот на руби на веб хуках

репозиторий публичный?

нет, там бот про сиськи