Skip to content

Clarify challenge-response nonce encoding#78

Open
cowbon wants to merge 2 commits into
veraison:mainfrom
cowbon:nonce
Open

Clarify challenge-response nonce encoding#78
cowbon wants to merge 2 commits into
veraison:mainfrom
cowbon:nonce

Conversation

@cowbon

@cowbon cowbon commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • specify challenge-response nonces as URL-safe base64
  • update examples and schemas to use a URL-safe base64 pattern

Testing

  • make check (api/challenge-response)

Signed-off-by: Ian Chin Wang <ian.chin.wang@oracle.com>

@yogeshbdeshpande yogeshbdeshpande left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@cowbon

cowbon commented Jul 14, 2026

Copy link
Copy Markdown
Author

Hi @yogeshbdeshpande , does this mean we can also proceed with veraison/services#428?

Comment thread api/challenge-response/README.md

@thomas-fossati thomas-fossati left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, just a small nit/suggestion.

Comment thread api/challenge-response/schemas/components.yaml Outdated
Comment thread api/challenge-response/challenge-response.yaml Outdated
Signed-off-by: Ian Chin Wang <ian.chin.wang@oracle.com>
@cowbon

cowbon commented Jul 21, 2026

Copy link
Copy Markdown
Author

LGTM, just a small nit/suggestion.

Thanks @thomas-fossati for pointing out.

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.

4 participants