livekit / protocol

LiveKit protocol. Protobuf definitions for LiveKit's signaling protocol
https://docs.livekit.io
Apache License 2.0
71 stars 61 forks source link

Add Twirp interceptor that prints curl commands #702

Closed dennwc closed 4 months ago

dennwc commented 4 months ago

Add Twirp interceptor that prints curl commands for each request made. It will be then used in our CLI.

changeset-bot[bot] commented 4 months ago

⚠️ No Changeset found

Latest commit: dcd288e97596ea8fa1140980679fe320fdd58bdc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR ``` Some errors occurred when validating the changesets config: The package or glob expression "github.com/livekit/protocol" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch. ```
dennwc commented 4 months ago

@davidzhao I was thinking about this too. Maybe it would be useful for lkctl, but apart from that ti could be moved to livekit-cli instead of the protocol. The only reason I put it here is because it does depend on the protocol used (Twirp).