-
-
Save bigbes/2930790181d1fe2a9dd26a484b617f35 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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