Skip to content
This repository has been archived by the owner on Sep 25, 2020. It is now read-only.

Implement tcat #92

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Implement tcat #92

wants to merge 2 commits into from

Conversation

kriskowal
Copy link
Contributor

Taking @benfleis’s work, I’ve swapped out the command parser with SHON (death to minimism) and added support for tcat subcommands. Thus, tcat 127.0.0.1:8888 service method -- jq .property' is now a thing you can do.

As needed, we can talk about:

  • support for Hyperbahn client
  • support for Thrift
  • support for multiple endpoints
  • support for readline interactive streaming
  • support for raw TTY interactive streaming, a.k.a., GAPING_SECURITY_HOLE, or simply, COOL_DEMO
  • support for a prefork pool

r @malandrew @Raynos @ShanniLi

stream.setEncoding('utf-8');
stream.on('data', onData);
stream.on('end', onEnd);
stream.on('eerror', onError);
Copy link
Contributor

Choose a reason for hiding this comment

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

eerror

@kriskowal
Copy link
Contributor Author

This is alas missing the release boat, but we will circle back and get it included.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants