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

Update wwwverifier to recent CD consultation #357

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

suzannajiwani
Copy link
Contributor

@suzannajiwani suzannajiwani commented Aug 21, 2023

Modified build scripts to make wwwverifier a subproject of the Identity Credential project to reduce copy-pasted code. Updated OriginInfo to match 18013-7 WD6 updates.

Tested appholder changes manually with unattended presentations via the wwwverifier as well as attended presentations with appverifier.

Fixes #350

  • [ X] Tests pass

@suzannajiwani suzannajiwani requested a review from davidz25 August 21, 2023 03:24
@suzannajiwani suzannajiwani marked this pull request as ready for review August 21, 2023 03:24
@suzannajiwani suzannajiwani force-pushed the issue-350 branch 2 times, most recently from 64c9904 to c3474e6 Compare August 30, 2023 19:04
Copy link
Contributor

@davidz25 davidz25 left a comment

Choose a reason for hiding this comment

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

Small nits but overall looks good!

@suzannajiwani suzannajiwani changed the title Update wwwverifier to WD6 Update wwwverifier to recent CD consultation Sep 6, 2023
@suzannajiwani suzannajiwani force-pushed the issue-350 branch 2 times, most recently from 9105f3c to 4ac5aa4 Compare September 11, 2023 21:50
Copy link
Contributor

@davidz25 davidz25 left a comment

Choose a reason for hiding this comment

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

Just a couple of small nits, almost ready to go!

generator.setOriginInfos(mReverseEngagementOriginInfos);
if (mReverseEngagementOriginInfos.size() != 0) {
generator.setOriginInfos(mReverseEngagementOriginInfos);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's valid to call this with an empty list, no? IOW, I think the check is superfluous...

Modified build scripts to make wwwverifier a subproject of the
Identity Credential project to reduce copy-pasted code. Updated
OriginInfo to match 18013-7 recent CD consultation.

Tested appholder changes manually with unattended presentations
via the wwwverifier as well as attended presentations with
appverifier.
@davidz25 davidz25 merged commit 034099e into openwallet-foundation-labs:master Sep 19, 2023
2 checks passed
@suzannajiwani suzannajiwani deleted the issue-350 branch September 20, 2023 18:22
@suzannajiwani suzannajiwani restored the issue-350 branch January 22, 2024 21:47
@suzannajiwani suzannajiwani deleted the issue-350 branch November 20, 2024 16:29
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.

Update the wwwverifier
2 participants