Skip to content

fix: seed-k8s.sh broken by EDC 0.15.1 key generation API change#583

Closed
Copilot wants to merge 9 commits intomainfrom
copilot/bump-edc-to-0-15-1
Closed

fix: seed-k8s.sh broken by EDC 0.15.1 key generation API change#583
Copilot wants to merge 9 commits intomainfrom
copilot/bump-edc-to-0-15-1

Conversation

Copy link

Copilot AI commented Mar 23, 2026

EDC 0.15.1 requires an explicit curve parameter when generating EC/EdDSA keys. seed-k8s.sh omitted it, causing a NullPointerException: The curve must not be null on participant context creation — leaving DID documents unpublished and all DID resolution returning HTTP 204, which cascades into complete E2E test failure.

Changes

  • seed-k8s.sh — consumer participant: Add "curve": "P-256" to EC keyGeneratorParams
 "keyGeneratorParams": {
-  "algorithm": "EC"
+  "algorithm": "EC",
+  "curve": "P-256"
 }
  • seed-k8s.sh — consumer STS secret: Capture clientSecret from the participant context creation response and store it in the consumer connector's vault under did:web:consumer-identityhub%3A7083:consumer-sts-client-secret — required for the connector to authenticate with the STS token endpoint (EDC_IAM_STS_OAUTH_CLIENT_SECRET_ALIAS)

Outstanding

The following identical fixes still need to be applied:

  • Provider participant: "curve": "P-256" + clientSecret vault storage
  • Issuer participant: "curve": "Ed25519" for EdDSA + missing x-api-key header on its curl call

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • central.sonatype.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED --add-opens=java.base/java.time=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:+HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI and others added 7 commits March 23, 2026 08:46
Copilot AI changed the title build: bump EDC to version 0.15.1 fix: seed-k8s.sh broken by EDC 0.15.1 key generation API change Mar 23, 2026
@ndr-brt ndr-brt closed this Mar 23, 2026
@ndr-brt ndr-brt deleted the copilot/bump-edc-to-0-15-1 branch March 23, 2026 16:01
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.

2 participants