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.

Sign up for GitHub

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

Jump to bottom

breaking change: added credential_status_detail in the status assertion #432

Merged

peppelinux merged 7 commits into versione-corrente from iss-401

Oct 16, 2024

Collaborator

m-basili commented Oct 1, 2024

This PR solves #401.

m-basili added 2 commits

September 30, 2024 17:00


          feat: Added credential_status_detail in the status assertion

f38822e


          feat: added description claim in the credential_status_detail

434c7aa

m-basili added this to the 0.8.1 milestone

m-basili requested review from grausof, peppelinux and fmarino-ipzs

October 1, 2024 11:54

peppelinux reviewed

View reviewed changes

docs/en/pid-eaa-entity-configuration.rst Outdated Show resolved Hide resolved

peppelinux reviewed

View reviewed changes

docs/en/pid-eaa-entity-configuration.rst Outdated Show resolved Hide resolved

peppelinux reviewed

View reviewed changes

docs/en/revocation-lists.rst Outdated Show resolved Hide resolved

peppelinux reviewed

View reviewed changes

docs/en/revocation-lists.rst Outdated Show resolved Hide resolved

peppelinux reviewed

View reviewed changes

examples/ec-eaa.json Outdated Show resolved Hide resolved

peppelinux approved these changes

View reviewed changes

Member

peppelinux left a comment

@m-basili please accept my editorials.

m-basili and others added 2 commits

October 1, 2024 17:06


          Apply suggestions from code review

3bbf6ac

Co-authored-by: Giuseppe De Marco <giuseppe.demarco@teamdigitale.governo.it>


          Update docs/en/pid-eaa-entity-configuration.rst

471d65f

Co-authored-by: Giuseppe De Marco <giuseppe.demarco@teamdigitale.governo.it>

grausof approved these changes

View reviewed changes

peppelinux reviewed

View reviewed changes

docs/en/revocation-lists.rst Outdated Show resolved Hide resolved

peppelinux reviewed

View reviewed changes

docs/en/revocation-lists.rst Outdated Show resolved Hide resolved


          Apply suggestions from code review

41ef979

Co-authored-by: Giuseppe De Marco <giuseppe.demarco@teamdigitale.governo.it>

peppelinux requested changes

View reviewed changes

Member

peppelinux left a comment

Any new proposal about ietf oauth status assertion MUST be developed in the related I-D, here:

https://github.com/peppelinux/draft-demarco-oauth-status-assertions

fmarino-ipzs approved these changes

View reviewed changes

grausof approved these changes

View reviewed changes

Collaborator Author

m-basili commented Oct 8, 2024

waiting for peppelinux/draft-demarco-oauth-status-assertions#87 approval.

peppelinux modified the milestones: 0.8.1, 0.9.0

m-basili added 2 commits

October 10, 2024 13:53


          feat: update credential_status_validity e credential_status_detail

f759636


          Merge branch 'versione-corrente' into iss-401

ee079fe

m-basili requested review from fmarino-ipzs, grausof and peppelinux

October 10, 2024 12:01

grausof approved these changes

View reviewed changes

peppelinux approved these changes

View reviewed changes

peppelinux merged commit 965f2e3 into versione-corrente

8 of 9 checks passed

peppelinux changed the title ~~Added credential_status_detail in the status assertion~~ breaking change: added credential_status_detail in the status assertion

peppelinux added the breaking change label

grausof mentioned this pull request

[Credential Issuer] Additional details on the released status assertion #401

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change