Skip to content

Add web-eid-1.1 token support - #132

Open
aarmam wants to merge 2 commits into
mainfrom
web-eid-mobile
Open

aarmam wants to merge 2 commits into
mainfrom
web-eid-mobile

Conversation

@aarmam

@aarmam aarmam commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: Sander Kondratjev sander.kondratjev@nortal.com
Co-authored-by: Mart Aarma mart.aarma@nortal.com

NFC-46

Signed-off-by: Sander Kondratjev <sander.kondratjev@nortal.com>
Co-authored-by: Mart Aarma <mart.aarma@nortal.com>
@aarmam
aarmam requested a review from mrts September 9, 2026 14:13
if (!matcher.matches()) {
return null;
}
final int major = Integer.parseInt(matcher.group(1));

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Safe to dismiss due to regex pattern validation?

return null;
}
final int major = Integer.parseInt(matcher.group(1));
final int minor = matcher.group(2) == null ? 0 : Integer.parseInt(matcher.group(2));

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Safe to dismiss due to regex pattern validation?

…ation

Add a short section to the library and example READMEs that lists only what
an application with a working regular Web eID integration has to change or
add for the mobile authentication flow, with links to the existing sections
and example classes that implement each step.

Signed-off-by: Mart Aarma <mart.aarma@nortal.com>
Co-authored-by: Sander Kondratjev <sander.kondratjev@nortal.com>
@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants