-
Notifications
You must be signed in to change notification settings - Fork 90
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 gRPC support for APK #2092
Add gRPC support for APK #2092
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2092 +/- ##
==========================================
- Coverage 27.93% 27.11% -0.83%
==========================================
Files 124 124
Lines 14597 15040 +443
Branches 2681 2681
==========================================
Hits 4078 4078
- Misses 10388 10829 +441
- Partials 131 133 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
test/integration/integration/tests/resources/tests/grpc-api.yaml
Outdated
Show resolved
Hide resolved
...2.apk.enforcer.commons/src/main/java/org/wso2/apk/enforcer/commons/model/ResourceConfig.java
Outdated
Show resolved
Hide resolved
gateway/enforcer/org.wso2.apk.enforcer/src/main/java/org/wso2/apk/enforcer/api/APIFactory.java
Show resolved
Hide resolved
generalize testcase a bit more generalize testcase a bit more generalize testcase a bit more generalize testcase a bit more generalize testcase a bit more generalize testcase a bit more
@@ -80,16 +80,15 @@ type APISpec struct { | |||
Sandbox []EnvConfig `json:"sandbox"` | |||
|
|||
// APIType denotes the type of the API. | |||
// Possible values could be REST, GraphQL, Async | |||
// Possible values could be REST, GraphQL, Async, GRPC etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have to go for alpha3 version. because alpha2 is already released
Purpose