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

Gestão de SSO delegada ao cliente da API #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cristiano2lopes
Copy link

@cristiano2lopes cristiano2lopes commented May 6, 2020

Este PR dá resposta ao problema indicado no issue #11.
Fa-ló expondo o método setSSO ao consumidor da API. Este fica responsável pela gestão do SSO.

Haveria claramente outras forma de atacar o problema, não expondo o método, por exemplo reproduzindo o conceito de sessão de assinatura utilizado na assinatura de PDFs em batch.

Esta solução deixa mais responsabilidade ao caller mas é mais versátil e dispensa código especifico para suportar o comportamento dependendo do artefacto a ser assinado (ficheiros pdf, hashes etc).

Do ponto de vista prático pessoal, não me forçou a alterar os ficheiros .i, dado que soluções alternativas obrigariam por certo à definição de novas bridges de tipos de dados. Para isso teria de levar a cabo uma aprendizagem sobre SWIG para a qual não tenho tempo neste momento (embora pareça interessante).

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

Successfully merging this pull request may close these issues.

1 participant