Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

✨feat: Suporte login UNB #36

Open
2 tasks done
Suyannesara opened this issue Mar 7, 2023 · 4 comments
Open
2 tasks done

✨feat: Suporte login UNB #36

Suyannesara opened this issue Mar 7, 2023 · 4 comments

Comments

@Suyannesara
Copy link

Suyannesara commented Mar 7, 2023

Quando instalo a api no meu projeto e passo minhas credenciais, recebo esse erro:

.../sigaa-login-ifsc.ts:93
throw new Error(this.errorInvalidCredentials);
              ^
Error: SIGAA: Invalid credentials.

Pontos importantes:

  • As minhas credenciais estão corretas;
  • Verifiquei que o padrão de requisição do IFSC é aparentemente o mesmo da UNB;

Decidi clonar o repositório para testar os examples importando a class Sigaa diretamente dos arquivos do projeto:

//examples/get-account-info.js
const { Sigaa } = require('../dist/sigaa-main.js');

const sigaa = new Sigaa({
  url: 'https://sigaa.unb.br'
});

Dessa maneira colocando minhas credenciais, funcionou perfeitamente e meus dados foram trazidos.

Ao rodar o examples/search-teachers.js, só que dessa vez utilizando const { Sigaa } = require('sigaa-api'); , chega a me retornar o resultado da pesquisa de professores com todos os dados, mas ao fim ainda joga o mesmo erro de SIGAA: Invalid credentials, o que é interessante porque nesse sample e no método de search em si as credenciais de login não são requeridas.

Não sei se ajuda, mas ao entrar no portal essas são as requisições feitas e os status retornados:
image

Não consegui identificar a solução, mas eu espero que esses debugs ajudem a descobrir

@Suyannesara Suyannesara changed the title Suporte login UNB ✨feat: Suporte login UNB Mar 7, 2023
@dduartee
Copy link
Contributor

dduartee commented Mar 8, 2023

Olá, fiz os mesmos passos e de fato, ele dá o erro citado.

EU acredito que seja alguma diferença no código do NPM em relação ao desse aqui direto do repositório. Testei instalando um meu fork do código, e funciona.

Então, por enquanto para não atrapalhar o seu desenvolvimento do código, adicione diretamente o repositório no package.json:

Com as aspas!

yarn add "https://github.com/GeovaneSchmitz/sigaa-api/"

ou com o npm

npm install "https://github.com/GeovaneSchmitz/sigaa-api/"

Preciso ver com o Geovane, acredito que por causa do ultimo commit (837a065) ser o merge de um pull request (#27) o npm acabou não publicando, e que por algum motivo dá esse erro.

@dduartee
Copy link
Contributor

dduartee commented Mar 8, 2023

Aproveitando a ocasião, acho que é interessante criar uma classe de login específica da UNB, seguindo o padrão:

const sigaa = new Sigaa({
  url: 'https://sigaa.unb.br',
  institution: 'UNB'
}); 

Para CASO tenha alguma diferença entre o SIGAA do IFSC para UNB pode ser modificado.

@dduartee
Copy link
Contributor

dduartee commented Mar 9, 2023

Decidi comparar a diferença das duas publicações e sim, está diferente.

No arquivo dist/sigaa-main.js aparenta ter um trecho de código que não deveria estar ali
image
onde normalmente se acabava no exports.Sigaa = Sigaa;, continua com um código de exemplo.
Verifiquei os commits anteriores e não encontrei nada que poderia injetar esse exemplo na classe principal.

Como no código typescript não há nada que injete o código, acredito que se fizer um commit alterando a versão do package.json para 1.0.35 deve resolver.

@dduartee dduartee mentioned this issue Mar 12, 2023
2 tasks
@dduartee
Copy link
Contributor

Decidi comparar a diferença das duas publicações e sim, está diferente.

No arquivo dist/sigaa-main.js aparenta ter um trecho de código que não deveria estar ali
image
onde normalmente se acabava no exports.Sigaa = Sigaa;, continua com um código de exemplo.
Verifiquei os commits anteriores e não encontrei nada que poderia injetar esse exemplo na classe principal.

Como no código typescript não há nada que injete o código, acredito que se fizer um commit alterando a versão do package.json para 1.0.35 deve resolver.

@GeovaneSchmitz

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants