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

Add ability to use Buf Schema Registry as a schema source for gRPC requests #6975

Merged
merged 10 commits into from
Feb 5, 2024

Conversation

srikrsna-buf
Copy link
Contributor

@srikrsna-buf srikrsna-buf commented Jan 9, 2024

As of now, gRPC requests support gRPC Reflection and parsing the protobuf files to get the schema. Buf Schema Registry (BSR) is a registry for storing and distributing protobuf sources. It also houses several popular open source protobuf definitions. This PR adds the ability to retrieve the schema from the BSR.

Screenshot 2024-01-23 at 4 22 34 PM

@CLAassistant
Copy link

CLAassistant commented Jan 9, 2024

CLA assistant check
All committers have signed the CLA.

@jackkav
Copy link
Contributor

jackkav commented Jan 11, 2024

looks good so far
image

  • input box doesn't let me type anything
  • the checkbox alignment is a little high.
  • rename the grpcRequest object reflectionApi rather than bufReflectionApi

thanks.

@srikrsna-buf
Copy link
Contributor Author

srikrsna-buf commented Jan 23, 2024

@jackkav Ready for another review, here's the updated screenshot:

Screenshot 2024-01-23 at 4 22 34 PM

jackkav
jackkav previously approved these changes Jan 25, 2024
@jackkav jackkav enabled auto-merge (squash) February 5, 2024 09:53
@jackkav jackkav merged commit 302172d into Kong:develop Feb 5, 2024
10 checks passed
jackkav added a commit to jackkav/insomnia that referenced this pull request Mar 13, 2024
…quests (Kong#6975)

* Add support for Buf Reflection Api

* Add test; Change tooltips to links

* style

* Remove label class

* request tests

* Update copy

* Rename prop; Fix alignment, input

* Add user agent header

* use onBlur and simplify

* fix lint

---------

Co-authored-by: jackkav <jackkav@gmail.com>
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

Successfully merging this pull request may close these issues.

None yet

4 participants