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

Web eid compat changes #14

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

Web eid compat changes #14

wants to merge 11 commits into from

Conversation

Jyrno42
Copy link
Contributor

@Jyrno42 Jyrno42 commented Jun 2, 2023

  • Added: LegacyIdentificationManager is now exported from the helper library too. This is to be used when interfacing with the deprecated actions based api of django-esteid

  • Changed: getCertificate now returns DER encoded certificate instead of its HEX representation. This is due to the change in the upstream extension.

  • Changed: IDCardManager's certificate is now a DER encoded certificate string instead of a certificate instance with encoded/hex fields.

  • Added: IDCardManager now has a new instance field supportedSignatureAlgorithms which is populated with the supported signature algorithms from the certificate during a signing session (its available after a successful call to getCertificate).

  • Changed: IdentificationManager.getError now expects the entire error object as its argument instead of only the error message value

  • Changed: Build system is now esbuild instead of webpack 4

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