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

[agents] Agent name should be equal to file name #19

Open
blackyblack opened this issue Aug 27, 2015 · 3 comments
Open

[agents] Agent name should be equal to file name #19

blackyblack opened this issue Aug 27, 2015 · 3 comments

Comments

@blackyblack
Copy link

When file name not equal to agent registered name agent cannot register in SuperNET.

Working call:

./BitcoinDarkd SuperNET '{"plugin":"SuperNET","method":"install","path":"/home/andrew/dev/snagentj/echodemo","daemonize":1,"timeout":5000}'

Not working call:

./BitcoinDarkd SuperNET '{"plugin":"SuperNET","method":"install","path":"/home/andrew/dev/snagentclj/run.sh","daemonize":1,"timeout":5000}'

It will be an issue for Windows users since filename should have extension to be executed (echodemo file does not start on Windows - echodemo.exe does)

@blackyblack
Copy link
Author

More info here:

Not working path:

./BitcoinDarkd SuperNET '{"plugin":"SuperNET","method":"install","path":"/home/andrew/dev/snagentclj/run.sh","daemonize":1}'

In btcd logs:

call_SuperNET_JSON
register.(echodemo)
recv: {"requestType":"instances","instanceids":["14761106664701345456", "14761106664701345456"],"daemonid":"3597766287535","instanceid":"14761106664701345456"}

Trying to call method:

./BitcoinDarkd SuperNET '{"plugin":"echodemo","method":"echo","echostr":"test"}'

Answer:

{"error":"cant find plugin"}

If we rename run.sh to echodemo it is working fine.

@jl777
Copy link
Owner

jl777 commented Sep 21, 2015

the name of the agent needs to match the internal name. so if renaming the file fixes this, maybe we can close this issue?

@blackyblack
Copy link
Author

Windows requires file extension to execute it properly. So better not rely on filename to maintain compatibility with another OS.

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

No branches or pull requests

2 participants