project-receptor / python-receptor

Project Receptor is a flexible multi-service relayer with remote execution and orchestration capabilities linking controllers with executors across a mesh of nodes.
Other
32 stars 21 forks source link

Unclear response when sending bad directive #27

Closed slemrmartin closed 5 years ago

slemrmartin commented 5 years ago

when sending this directive:

receptor send --socket-path=/tmp/receptor.sock --directive=receptor-http.execute node-a '{"method": "GET", "url": "http://localhost:3002/api/sources/v1.0/sources"}'

this response in returned:

{"message_id": "644cdb7c-384c-4f33-bfe7-e5c8d8acebe3", "sender": "node-a", "recipient": "controller", "message_type": "response", "timestamp": "2019-09-18T13:30:23.494909", "raw_payload": "not enough values to unpack (expected 2, got 1)", "directive": null, "in_response_to": "d906defc-0699-4f98-8108-a416b1523c52", "ttl": 15, "serial": 2, "code": 1}

debug log from node:

DEBUG 2019-09-18 15:32:59,823 protocol b'{"frame_id": "eb388e83-45fa-4756-824f-d31e892cefa6", "sender": "controller", "recipient": "node-a", "route_list": ["controller", "controller"], "inner": "{\\"message_id\\": \\"e8659027-8b6c-4329-8483-dc224d7c8cdd\\", \\"sender\\": \\"controller\\", \\"recipient\\": \\"node-a\\", \\"message_type\\": \\"directive\\", \\"timestamp\\": \\"2019-09-18T13:32:59.807935\\", \\"raw_payload\\": \\"{\\\\\\"method\\\\\\": \\\\\\"GET\\\\\\", \\\\\\"url\\\\\\": \\\\\\"http://localhost:3002/api/sources/v1.0/sources\\\\\\"}\\", \\"directive\\": \\"receptor-http.execute\\", \\"in_response_to\\": null, \\"ttl\\": null, \\"serial\\": 1, \\"code\\": 0}"}\x1b[K'
DEBUG 2019-09-18 15:32:59,886 router Shortest path to controller with cost 1 is ['controller', 'node-a']
DEBUG 2019-09-18 15:32:59,887 router Sending 351bdef4-c45d-4e44-a3dc-80f3ea4e355c to controller via controller
DEBUG 2019-09-18 15:32:59,887 router Forwarding frame e720601f-ce93-4dbe-add9-75f383c4b17a to controller

The --directive should be "receptor_http:execute" instead of "receptor-http.execute"

slemrmartin commented 5 years ago

cc @matburt

matburt commented 5 years ago

This is fixed up here: https://github.com/project-receptor/receptor/pull/28