Skip to content
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

extract config, and add working config from patchbay #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mixmix
Copy link
Member

@mixmix mixmix commented Jan 9, 2019

scuttle-shell wasn't quite working for me. When I went to investigate, I noticed the config was a bit odd compared to patchbay. I'm still getting my head around this connections config stuff.

But I've found using this branch in patchbay everything works happily (that I've seen so far!)

In this PR:

  • extraction of config building into ./config.js
  • 🔥 dropping minimist in config building because ssb-config uses RC, which I think does everything we want for loading config?! this needs attention
  • copies in connections config which works in patchbay

@mixmix
Copy link
Member Author

mixmix commented Jan 9, 2019

I see one of these socket errors (where previously I'd see 5+), but patchbay seems to run fine

err? { Error: connect ECONNREFUSED /home/mixmix/.ssb/socket
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '/home/mixmix/.ssb/socket' }

// const minimist = require('minimist')

module.exports = function buildConfig ({ appname }) {
// mix: note ssb-config uses RC, which loads config opts from process.argv
Copy link
Member Author

@mixmix mixmix Jan 9, 2019

Choose a reason for hiding this comment

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

hmmm on reading docs, I don't think RC actually loads things from process.argv ):

So this code should be reinstated. However, I still don't know what the last line does :

  argv = ~i ? argv.slice(0, i) : argv 

@cryptix
Copy link
Member

cryptix commented Jan 17, 2019

Sorry that I can’t add anything here. The RC vs JSON config is black magic that always confused me... mostly just that we have two config file systems to begin with.

Re the socket error is fine as long as it works in the end. The Client tries to redial until it works. The fluctuation you see should just be difference in sbot boot time before it can accept connections.

@stale
Copy link

stale bot commented May 3, 2019

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@stale stale bot added the stale label May 3, 2019
@christianbundy
Copy link
Contributor

Anything I can do to help with this? Not sure if scuttle-shell is still active, but noticed this PR was still open.

@stale stale bot removed the stale label Nov 19, 2019
@stale
Copy link

stale bot commented Feb 17, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@stale stale bot added the stale label Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants