Skip to content

Instantly share code, notes, and snippets.

@bigbes
Created September 8, 2017 13:25
Show Gist options
  • Save bigbes/2930790181d1fe2a9dd26a484b617f35 to your computer and use it in GitHub Desktop.
Save bigbes/2930790181d1fe2a9dd26a484b617f35 to your computer and use it in GitHub Desktop.
bauth.lua
style:
* Форматировать не помещающиеся на одну строку массивы как в bauth.lua:31
(например bauth.lua:71)
* Никаких пробелов в конце строки
* в случае массивов на одной строке - оставлять пробел после первой и до послей
скобки
* стоит избегать использование {} вместо ({}) в вызовах функций. Кроме
сушествующих идиом (box.cfg{}/ffi.cdef[[]])
```
return {
status = 200,
headers = { ['content-type'] = 'application/json' },
body = json.encode({ token = user_token })
}
```
* string_starts -> string.startswith (уже есть в тарантуле)
* admin_handler -> с помощью HTTP методов ты можешь разделить роуты для разных
типов запросов (дабы избежать кучи if/elseif/else)
* не обязательно указывать `headers = { ['content-type'] = 'application/json' },`
в теле ответа, стоит использовать render({ json = true })
* разделить auth_wrapper на две составляющие: декоратор для врапа с self и
декоратор для аутентификации. первый я бы запилил в виде функтора. и
переиспользовал бы в будущем.
* в отписывании ошибки нужно больше информации. предлагаю завести общую функцию,
которая по запросу будет пилить шаблонный log.error + сообщение пользователя
* log.verbose для каждого запроса
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment