These are chat archives for synrc/n2o

10th
Nov 2016
PlatinumThinker
@platinumthinker
Nov 10 2016 10:23
обновил n2o с версии 0.9 до 3.4 всё заработало. только спустя 10 минут сессия отваливается - в n2o данные летят, а мне ничего..
есть значение timer по дефолту {0, 10, 0}, после которого дропаются данные из ets, есть ttl в N минут который почему то не обновляется
PlatinumThinker
@platinumthinker
Nov 10 2016 10:29
при том сессия отваливается даже если ей пользоваться, т.е. ходят eventы от нажатия кнопок
это вообще нормально?
Andy
@m-2k
Nov 10 2016 11:33
а ты бы хотел дергать expire time в ETS для сессии после каждой активности от пользователя?
выстави в конфиге пару дней и всё
PlatinumThinker
@platinumthinker
Nov 10 2016 12:47
тут палка о двух концах - с одной стороны resource leak, с другой недовольные пользователи
прикручу завершение сессии хотя бы
чтобы на login page юзер попадал
Andy
@m-2k
Nov 10 2016 12:49

чтобы на login page юзер попадал

у меня так и есть

где у тебя там лики? в голове только?
PlatinumThinker
@platinumthinker
Nov 10 2016 12:50
ну проблема как раз в том что n2o никак меня не уведомляет что сессия завершилась
Andy
@m-2k
Nov 10 2016 12:50
миллион записей пихни в ETS и посмотри сколько это в гигабайтах
n2o и не должно
ваш redis новомодный тоже не уведомляет
PlatinumThinker
@platinumthinker
Nov 10 2016 12:51
он не наш
Andy
@m-2k
Nov 10 2016 12:51
это слишком накладно
PlatinumThinker
@platinumthinker
Nov 10 2016 12:51
не юзал
Andy
@m-2k
Nov 10 2016 12:51
зря
PlatinumThinker
@platinumthinker
Nov 10 2016 12:52
мне норм
Andy
@m-2k
Nov 10 2016 12:52
не льсти себе
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 17:20
Hi
in nitrogen element textbox
value is js_escape
this cause troubles when using the same code to display in html tag
I would let it be done at the application level
what do you think ?
Roman
@AstRonin
Nov 10 2016 17:29
Maybe remove escape js code from function js_escape...
Roman
@AstRonin
Nov 10 2016 18:10
tag
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 18:38
Ok I found a workaround right now
i'm using
data_fields= [{value, Tuple#tb_usergroup.name}]
and then value property is not escaped and I can re-use my code
I prefer trying to stick to syncr basecode instead of forking my own
In my example, tb_usergroup.name was already html encoded and js_escaped, and using the legacy value field would make it escaped twice !
Namdak Tonpa
@5HT
Nov 10 2016 18:51
any unmerged pull requests ?
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 18:57
what do you mean ?
you want me to change the current value behaviour ? that could break some existing code
Namdak Tonpa
@5HT
Nov 10 2016 18:58
you can always plug application:get_env(nitro,render_mode,custom_flag)
and turn breakage to compatibility flag
if you really need it
we removed all js_escaping from system code
as it was in nitrogen originally
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 18:59
a few nitro elements have their value entry js_escape right now !
Namdak Tonpa
@5HT
Nov 10 2016 18:59
they shouldn't
which one?
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 19:00
textbox, input for example
  {<<"value">>,      nitro:js_escape(Record#input.value)},
Namdak Tonpa
@5HT
Nov 10 2016 19:01
hm
it shouldn't be like that
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 19:01
I agree
Namdak Tonpa
@5HT
Nov 10 2016 19:01
just remove it
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 19:01
It breaks my code :-)
do you want to remove what is on event field as well ?
such as onkeyup, etc...
there are quite a few more ....nitro:js_escape
It should be on the hands of the coder to provide proper code I think and It would be more consistent
what do you think ?
Namdak Tonpa
@5HT
Nov 10 2016 19:04
I totally agree
I will accept total remove js_escape from all the nitro elements
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 19:05
good I will do that
Namdak Tonpa
@5HT
Nov 10 2016 19:07
feel free to do it with nitro any time )
just push it to master
Sébastien Saint-Sevin
@seb3s
Nov 10 2016 19:08
ok thanks