-
Notifications
You must be signed in to change notification settings - Fork 1
EUDIPLO's presentation encryption algorithm is not supported #951
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
Open
Magnus-Kuhn
wants to merge
39
commits into
release/openid4vc
Choose a base branch
from
update-eudiplo
base: release/openid4vc
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
ca7c54a
test: update eudiplo
Magnus-Kuhn 13d25e5
test: try another tenant
Magnus-Kuhn 1219fca
fix: allow AES128GCM
Magnus-Kuhn 0feb6b0
test: fix eudiplo tests
Magnus-Kuhn 03e8498
Merge branch 'release/openid4vc' into update-eudiplo
mergify[bot] c43ccf5
Merge branch 'release/openid4vc' into update-eudiplo
mergify[bot] a4dee56
test: check reading of enmeshed data
Magnus-Kuhn c896467
test: make tests work
Magnus-Kuhn 14d3f3d
test: add eudiplo pin test
Magnus-Kuhn fca803c
test: check display information
Magnus-Kuhn edbaeeb
test: update eudiplo assets and test to eudiplo
Magnus-Kuhn a808b53
test: update compose openid4vc
Magnus-Kuhn 141dfa4
chore: disable lint
Magnus-Kuhn 2959a18
chore: update dependencies
Magnus-Kuhn 6085847
chore: updates in compose
Magnus-Kuhn 03e0c17
chore: fix eudiplo config property names
tnotheis 8caf24b
chore: fix eudiplo base url
tnotheis 80f8c7e
chore: log error in ShareAuthorizationRequestRequestItemProcessor.accept
tnotheis e31534f
chore: remove obsolete configs
Magnus-Kuhn 71c062a
chore: remove mongodb dep
Magnus-Kuhn 6043619
refactor: rename to oid4vc service config
Magnus-Kuhn e8877ed
chore: remove unnecessary port mapping
Magnus-Kuhn a914816
test: remove eudiploContainer
tnotheis 2e2bb4b
chore: add docker compose dependencies to oid4vc-service
tnotheis f02ec74
chore: move environment variables to compose file
tnotheis 74bb306
test: remove unnecessary test service
Magnus-Kuhn 3c44244
Merge branch 'update-eudiplo' of https://github.com/nmshd/runtime int…
Magnus-Kuhn e6a52a5
fix: correct healthcheck
Magnus-Kuhn f8e7d52
test: correctly set env vars
tnotheis 1d65954
chore: upgrade oid4vc-service to 3.0.1
tnotheis 202eb36
test: make connector auto accept pending relationships
tnotheis 49130ff
fix: switch to localhost in tests
Magnus-Kuhn 29101a0
refactor: add comment
Magnus-Kuhn 7b5796f
test: disable eslint
Magnus-Kuhn 4f6ee9e
fix: don't change every test behavior
Magnus-Kuhn 79f1f64
chore: update dependencies
Magnus-Kuhn 67847b2
chore: clean npm install
Magnus-Kuhn 096de50
refactor: satisfy linter
Magnus-Kuhn 0e415b7
ci: exclude ajv vulnerability
Magnus-Kuhn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
Binary file not shown.
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,5 +48,8 @@ | |
| "ts-node": "^10.9.2", | ||
| "tsconfig-paths": "^4.2.0", | ||
| "typescript": "^5.9.3" | ||
| }, | ||
| "overrides": { | ||
| "tar": "^7.5.9" | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,6 +9,8 @@ import { LocalRequestInfo } from "../IRequestItemProcessor"; | |
|
|
||
| export class ShareCredentialOfferRequestItemProcessor extends GenericRequestItemProcessor<ShareCredentialOfferRequestItem> { | ||
| public override async canCreateOutgoingRequestItem(requestItem: ShareCredentialOfferRequestItem, _request: Request, _recipient?: CoreAddress): Promise<ValidationResult> { | ||
| if (process.env.TEST_ENVIRONMENT === "container") return ValidationResult.success(); // for the test scenario that this runs inside a container which can't resolve a localhost credential offer | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe we can use the network mode "host". See https://docs.docker.com/engine/network/drivers/host/ |
||
|
|
||
| const offer = await this.consumptionController.openId4Vc.resolveCredentialOffer(requestItem.credentialOfferUrl); | ||
|
|
||
| const preAuthorizedCodeGrant = offer.credentialOfferPayload.grants?.["urn:ietf:params:oauth:grant-type:pre-authorized_code"]; | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need this? It's dangerous to override peer dependencies, because it might introduce breaking changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To fix the vulnerability GHSA-83g3-92jg-28cx