Skip to content

Conversation

zachferland
Copy link
Contributor

Adds flag/config to support recon service api. Currently intended for testing/benchmarking env only (not backwards support, run with pubsub off, etc). Tries to minimize changes outside of recon service.

Add config

{
  ...
  "node": {
    "recon-url": "http://127.0.0.1:6001"
    ...
  },
  ...
}

References:
Spec - https://www.notion.so/threebox/Recon-js-ceramic-3964bec706434050ad6a80653cea990a
Rust Ceramic - ceramicnetwork/rust-ceramic#60

@zachferland zachferland force-pushed the feat/recon-rs-ceramic branch from 3120c92 to 7cf0883 Compare August 1, 2023 19:45
@oed
Copy link
Collaborator

oed commented Aug 2, 2023

Why are we using recon-url and not enable-recon with a boolean? I thought we wanted rust-ceramic to expose the same http server for ipfs (kubo) as well as recon?

Maybe this is not the case @nathanielc ?

@nathanielc
Copy link
Contributor

For technical reasons we were not able to expose both Kubo RPC and the Ceramic API on the same port. (They use different incompatible http frameworks) We can port Kubo RPC over to the framework Ceramic API uses but it wasn't a priority to start.

@oed
Copy link
Collaborator

oed commented Aug 2, 2023

Ok, thanks for the context @nathanielc. Seems like it might be worth to do that later when we want to migrate users to use rust-ceramic instead of kubo.

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