Fork me on GitHub
#clojure-russia
<
2016-04-23
>
dottedmag07:04:29

@niwinz: А расскажи, почему в https://github.com/funcool/buddy-sign/blob/master/src/buddy/sign/jws.clj#L158 :exp и :iat не рекомендуются? Я их использую, чтобы юнит-тестировать код, который создаёт токены.

dottedmag07:04:49

Мне проще создать токен, валидный с unix timestamp 0 в течение 3600 секунд, чем чесать репу, почему тесты иногда ломаются из-за того, что что-то долго тестировалось.

niwinz07:04:53

The additional options will be removed because them are redundant

niwinz07:04:08

you can pass the same claims in the token data

dottedmag07:04:35

А, дошло. Спасибо =)

dottedmag07:04:53

@niwinz: Я хочу добавить в unsign возможность указать "текущий таймштамп" вместо (util/timestamp). OK?

dottedmag07:04:53

Я могу, конечно, в тесте сделать with-redefs на util/timestamp, но это как-то криво выглядит.

niwinz07:04:40

at this moment, you can't specify your own timestamp

niwinz07:04:21

but I think we can provide a way to do that

dottedmag07:04:30

Ну, вот я и добавлю какой-нибудь ключ :now в опции, чтобы его указать.

dottedmag07:04:49

В общем, жди PR :)

niwinz08:04:00

@dottedmag: I have plans a release a new version of buddy-sign today because I have fixed one bug. I can add that parameter myself

niwinz08:04:23

if you have already started

niwinz08:04:27

then I'll wait

dottedmag08:04:33

@niwinz: Тогда втяни изменения с https://github.com/dottedmag/buddy-sign, плз

artemyarulin08:04:22

Тут постили про чувака который после 5 лет с кложурой начал искать альтернативу, его первый кандидат это хаскель simple_smile http://martintrojer.github.io/beyond-clojure/2016/04/21/beyond-clojure-haskell интересное чтиво на самом деле

dottedmag08:04:30

@niwinz: Я уже делаю поддержку :now, так что подожди, плз.

dottedmag08:04:33

Будет готово сегодня.

dottedmag09:04:29

@niwinz: Готово. Бонус номер 1: buddy.sign.jws-tests работают моментально. Бонус номер 2: багфикс валидации :exp. Бонус номер 3: валидация :iat.

dottedmag09:04:46

@niwinz: Бонус номер 4: поправил неправильный тест из Fix NPE on header parsing on jwe/jws. lein test надо запускать перед тем, как пушишь :)

dottedmag09:04:21

@niwinz: Проверь внимательно commit-per-commit, пожалуйста. Криптография всё-таки, как-никак.

lapooh09:04:53

Добрый день! Ктонить встречался с arity exception в yesql при задании простого запроса(defquery) и попытки его вызова?

niwinz10:04:26

@dottedmag: awesome! thanks for your time! For now, I need to go away, but this afternoon I will review that (and port some improvements to JWE)

leov10:04:56

всем привет

leov10:04:08

а как сделать аналог required key в руби?

leov10:04:37

def func required_foo:, required_bar:, optional_c: 3, optional_d: 5 .. end

leov11:04:12

спасибоу

leov13:04:46

а как в ринг хендлере скипнуть проверку цсрф токена в конкретном файле где объявлен маршрут

leov13:04:58

а не городить для этого огород в центральном хендлере

leov13:04:13

прописывая секьюрные и несекьюрные маршруты

seryh13:04:23

@abtv: непривычная структура у веб приложения твоего (tech-radar), сложно читать

abtv13:04:01

а что непривычного, можешь пояснить? @seryh

seryh13:04:41

@abtv: ну обычно сразу видно разделение на некие роутеры модельки контроллеры

seryh13:04:01

я обычно начинаю читать веб приложение с роутеров, смотрю куда приходят данные и что с ними происходит

abtv13:04:34

а, ну я по-другому разделил. там всего два маршрута, основная часть кода там - процессинг твитов, ну и некоторый boilerplate для Component

abtv13:04:23

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

abtv13:04:04

а у тебя нет примера как ты делаешь? @seryh

seryh13:04:32

да я в кложе еще джуниор ) на мои примеры смотреть не стоит

abtv13:04:14

тут есть еще один момент. тот же Боб Мартин предлагает не делить приложение на роутеры, контроллеры, модели, а делить на бизнес-сущности. по крайней мере делать так, чтобы из структуры приложения были видны эти бизнес-сущности

abtv13:04:23

у меня пока не получается)

seryh13:04:47

вообще хорошо читается такая классическая структура https://github.com/flyingmachine/gratefulplace/tree/ba8bd2dd1b7ec528f3fafeafd53b2ff5b3803f18/src/gratefulplace

seryh13:04:29

а к методологии Боба Мартина возможно мир еще не готов

abtv13:04:57

о, спасибо, посмотрю

abtv14:04:34

да, читается легко simple_smile

dottedmag14:04:39

@abtv: У дяди Боба говорится про приложения, где есть вменяемое количество осмысленного кода, а не 5% логики и 95% "сантехники".

abtv14:04:34

а ты можешь буквально пару минут уделить https://github.com/abtv/tech-radar ? запутанно? @dottedmag

abtv14:04:41

фронтенд корявый пока, вопрос только про бэкенд @dottedmag

dottedmag14:04:20

@abtv: Да вроде всё читабельно

abtv14:04:29

спасибо simple_smile

niwinz18:04:45

@dottedmag: deployed buddy-sign 0.13.0 with your changes 😉

dottedmag19:04:08

@niwinz: Спасибо.

niwinz19:04:12

tebe spasibo! 😉

dottedmag19:04:25

Вот так я внезапно и стал коммиттером #2 в кложурной библиотеке, хотя ничего рабочего на кложуре так и не написал ещё :)