-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: poc tweaks to get running (needs tidy up) #1
base: main
Are you sure you want to change the base?
Conversation
- ISSUER_PRE=${ISSUER_PRE:-EN-JeVj67PgehbEBD8W7W5aRJ8FZ9D2_WTaIlb5UprRL} | ||
- ISSUER_REGK=${ISSUER_REGK:-EEPQVQ6uMfF_aTHVXn8oo6DceQKkhNvIX9UulTg1Cx1G} | ||
- KERIA_ENDPOINT=${KERIA_ENDPOINT:-http://keria-g1:3901} | ||
- KERIA_BOOT_ENDPOINT=${KERIA_BOOT_ENDPOINT:-http://keria-g1:3903} |
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.
For me keria-g1
is on the same traefik network (commented out below). This should be cardanofoundation/cf-idw-keria:keria-3c72a12c8e8fea354b603d8406b31f6bab860a11
.
It doesn't need to be the same one as the issuer, whichever works.
ports: | ||
- 3000:3000 | ||
- 5632:3000 |
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.
port changes just as these were already open on the machine, ignore..
@@ -36,7 +36,8 @@ async function getClient(): Promise<SignifyClient> { | |||
|
|||
// If already resolved, this is very fast | |||
await waitAndGetDoneOp(client, await client.oobis().resolve(config.issuerOobi)); | |||
await waitAndGetDoneOp(client, await client.oobis().resolve(`http://localhost:3000/oobi/${config.schemaSaid}`)); | |||
// await waitAndGetDoneOp(client, await client.oobis().resolve(`http://server:3000/oobi/${config.schemaSaid}`)); | |||
await waitAndGetDoneOp(client, await client.oobis().resolve(`http://t10n.guild1.com:3000/oobi/EA3NRCtGF0czMPeiG5-CWbgCnmcpBDpPo2mYlxoGkk0j`)); |
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.
Maybe was some issue in the production build overriding VITE_BACKEND_BASE_URL
via the docker URL. Something to fix
@@ -11,7 +11,7 @@ enum BLOCKFROST_ASSETS_URL { | |||
PREVIEW = "https://preview.blockfrost.cf-systems.org" | |||
} | |||
|
|||
const BACKEND_BASE_URL = import.meta.env.VITE_BACKEND_BASE_URL ?? "http://localhost:3000"; | |||
const BACKEND_BASE_URL = import.meta.env.VITE_BACKEND_BASE_URL ?? "http://dev.keria.cf-keripy.metadata.dev.cf-deployments.org:5632"; |
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.
Same here. Whatever the case this should be the URL of the /server
deployment
console.error(error); // @TODO - check for actual 404 or re-throw | ||
}); | ||
// const credential = await fetch(`${BACKEND_BASE_URL}/credentials/${vci}`).catch(error => { | ||
// console.error(error); // @TODO - check for actual 404 or re-throw |
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.
The idea here was to check if it exists already and just return it if so, but fetch throws an error for 404 so I just commented it out. Would be easy to adjust.
//if (credential.ok) { | ||
// return await credential.json(); | ||
//} | ||
if (vci !== "EDlZ9u5_yGUiXWyNx2Tp-N4SXj5oRxF6KL8c3gMP7XbR") return; |
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.
I believe this ACDC is from an old version. Maybe this change can be reverted, from memory I think it was just noise in the logs when I was debugging the final issues.
No description provided.