micheldoumit / tripvise

0 stars 0 forks source link

header é um bom nome? #43

Closed marciotoze closed 9 years ago

marciotoze commented 9 years ago

achei header muito genérico pra um método que só poe uma autorização

estava pensando em algum nome focado em autorização, mas tive uma ideia melhor.

ao invés de você criar um helper de auth, o que acha de criar um helper pra header mais generico?

tipo

def header(*args)
  options = args.extract_options!
  token = 'Token token=' + options[:token]
  controller.request.headers['Authorization'] = token
end

dessa forma fica visivelmente melhor o que você esta passando na chamada do header

header(token: 'xxx')

e o metodo fica aberto pra receber possíveis atributos futuramente.

o que acha? ps: queria ouvir sua opinião nessa.

commit file reference
6ff2b8f3bc791e7acdb5f91def1c72dd1e7a2da4 spec/support/auth_helper.rb#L2 #38
MarcusSky commented 9 years ago

acho uma boa ideia! ficou bem mais claro o que o header tá fazendo :+1: