Skip to content

feat: switch to construct-x wallet in local testbed#6

Open
eschrewe wants to merge 20 commits into
developfrom
test/con-x-wallet
Open

feat: switch to construct-x wallet in local testbed#6
eschrewe wants to merge 20 commits into
developfrom
test/con-x-wallet

Conversation

@eschrewe
Copy link
Copy Markdown

WHAT

  • updated local docker compose in order to use the new con-x-wallet
  • updates to the bruno collection, including some improved automatic testing

WHY

  • use the new con-x-wallet

@eschrewe eschrewe requested a review from mhellmeier May 22, 2026 07:13
@F-Node-Karlsruhe
Copy link
Copy Markdown

The PR mainly replaces the eclipse identity hub by the identity hub fork of construct-x, the construct-x wallet for further development. Looks good to me

@mhellmeier mhellmeier changed the title Test/con x wallet feat: switch to construct-x wallet in local testbed Jun 3, 2026
Copy link
Copy Markdown
Member

@mhellmeier mhellmeier left a comment

Choose a reason for hiding this comment

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

Only minor changes. I tested the Docker-Compose file and the Bruno collection, and everything works as expected.

Comment on lines +58 to +61
# AES-Key erzeugen
aes_key="$(openssl rand -base64 32 | tr -d '\n')"

# AES-Key in Vault schreiben, Pfad an Prefix koppeln
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When commenting, please always use the English language.

}

post {
url: {{ISSUER_ISS_API}}/v1alpha/participants/{{ISS_PART_CONT}}/credentials/27fa4b9e-b15b-4908-a174-e440fc026e99/revoke
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The {{ISSUER_ISS_API}} is dynamic, but the credential ID is hardcoded (27fa4b9e-b15b-4908-a174-e440fc026e99).

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.

3 participants