Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add API Key support #9

Closed
ftonato opened this issue Jan 19, 2016 · 3 comments
Closed

Add API Key support #9

ftonato opened this issue Jan 19, 2016 · 3 comments

Comments

@ftonato
Copy link
Contributor

ftonato commented Jan 19, 2016

Conforme descrição do projeto:

IMPORTANTE: Para usar Vagalume API, é necessário que seu aplicativo tenha credenciais de autorização. Cadastre-se no Vagalume e crie sua chave de API. Esta credencial será requerida a partir do dia 23/11/2015.

Poderíamos fazer a atualização conforme indica a issue que explica como deve ser o novo padrão nas requisições.

Exemplo no método getNoticias()

PUBLIC.getNoticias = function () {
  return PRIVATE.wwwRequest('/news/index.js?apikey={'+apiKey+'}');
};

Algumas ideias de como receber esse valor:

  1. O método a ser usado podería receber esse valor. Acho que não é a melhor opção
  2. A função Vagalume podería receber esse valor var api = new Vagalume('valor_apiKey'); Acho que essa é a melhor opção

O que acha @gpedro ?

@gpedro
Copy link
Owner

gpedro commented Jan 19, 2016

olá @ftonato,

efetuei alguns testes de implementação do apikey e vi que essa história de VagalumeAPI não está nada instável :(

  • não passando está funcionando.
  • passando a key, eu caio em uma página do nginx o.O

de qualquer forma, tenho que dar relida na API novamente e adicionar os novos métodos (:

@ftonato
Copy link
Contributor Author

ftonato commented Jan 19, 2016

Estranho não?

Acabei de fazer alguns testes locais aqui e percebi que:
Os métodos getNoticias() e getRank() estão funcionando das duas formas (com o apiKey e sem o apiKey). 😨

Visto que nem todos os métodos estão aceitando o apiKey, podemos provisóriamente fazer cada método receber o parâmetro ou deixar implementado como sugeri anteriormente e fazer um tratamento nos métodos que não aceitam o apiKey.
Dessa forma, manteremos o suporte desde já, ainda que a API oficial não siga o que ela mesmo diz ( que seria necessário usar as credenciais de autorização)...

@gpedro o que acha?

@ftonato
Copy link
Contributor Author

ftonato commented Aug 17, 2017

Fala @gpedro, como o projeto parece não ser mais receber atualizações, encerrarei a issue por hora, mas poderemos voltar a discutir isso em um futuro...

Abraço,

@ftonato ftonato closed this as completed Aug 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants