Skip to content

Explicitly set credential transfer option#188

Merged
geonnave merged 2 commits intolake-rs:mainfrom
geonnave:explicit-credential-transfer-option
Jan 12, 2024
Merged

Explicitly set credential transfer option#188
geonnave merged 2 commits intolake-rs:mainfrom
geonnave:explicit-credential-transfer-option

Conversation

@geonnave
Copy link
Copy Markdown
Collaborator

@geonnave geonnave commented Jan 10, 2024

Addressing #187.

With this:

  1. no more need to pass information (kid or full credential) already present in cred_x (which already is in the state)
  2. the choice of how to transfer the credential becomes more clear

@geonnave geonnave changed the title Explicit set credential transfer option Explicitly set credential transfer option Jan 10, 2024
@geonnave
Copy link
Copy Markdown
Collaborator Author

geonnave commented Jan 11, 2024

@malishav since this changes the API can you give a quick look? (see commit af6cc25)

@geonnave geonnave merged commit c1a7f21 into lake-rs:main Jan 12, 2024
@geonnave geonnave deleted the explicit-credential-transfer-option branch January 12, 2024 10:36
@geonnave geonnave mentioned this pull request Jan 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

type:new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@geonnave