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

[management] Service Management Ecosystem (SME): API HTTP Connector Integration #6132

Closed
mouligno opened this issue Sep 9, 2021 · 4 comments · Fixed by gravitee-io/gravitee-node#86

Comments

@mouligno
Copy link
Contributor

mouligno commented Sep 9, 2021

Dependance with #6130 - current behavior moving to plugin so we would have a dynamic form

Expected behavior

As a developer, I want to use connector plugins to display more API backend options.

Configuration form updates

  • Some panels are collapsed :
    -- HTTP REQUEST
    -- HTTP CONFIGURATION
    -- HTTP PROXY
    -- SSL/TLS
    -- SSL-Client authentification
  • Panels are collapsed independently

Gravitee_io_Management-form

@mouligno mouligno added this to the APIM - 3.13.0 milestone Sep 9, 2021
@phiz71 phiz71 self-assigned this Sep 21, 2021
@mouligno mouligno changed the title [SME] API Connector Integration [SME] API HTTP Connector Integration Sep 28, 2021
@gcusnieux gcusnieux self-assigned this Oct 5, 2021
phiz71 pushed a commit to gravitee-io/gravitee-definition that referenced this issue Oct 6, 2021
phiz71 pushed a commit to gravitee-io/old-gravitee-api-management that referenced this issue Oct 6, 2021
Adapt APIM code to the new definition
Create services and resources in management API to expose available connectors
gravitee-io/issues#6132
phiz71 pushed a commit to gravitee-io/old-gravitee-api-management that referenced this issue Oct 6, 2021
bump gravitee-definition & gravitee-plugin version
Adapt APIM code to the new definition
Create services and resources in management API to expose available connectors

gravitee-io/issues#6132
gcusnieux added a commit to gravitee-io/gravitee-connector-http that referenced this issue Oct 7, 2021
gcusnieux added a commit to gravitee-io/gravitee-management-webui that referenced this issue Oct 7, 2021
gcusnieux added a commit to gravitee-io/gravitee-management-webui that referenced this issue Oct 7, 2021
gcusnieux pushed a commit to gravitee-io/old-gravitee-api-management that referenced this issue Oct 27, 2021
bump gravitee-definition & gravitee-plugin version
Adapt APIM code to the new definition
Create services and resources in management API to expose available connectors
Add connector-http in distribution pom

gravitee-io/issues#6132
@gcusnieux gcusnieux reopened this Oct 27, 2021
gcusnieux added a commit to gravitee-io/gravitee-connector-http that referenced this issue Oct 27, 2021
gcusnieux pushed a commit to gravitee-io/gravitee-definition that referenced this issue Oct 27, 2021
gcusnieux pushed a commit to gravitee-io/old-gravitee-api-management that referenced this issue Oct 27, 2021
bump gravitee-definition & gravitee-plugin version
Adapt APIM code to the new definition
Create services and resources in management API to expose available connectors
Add connector-http in distribution pom

gravitee-io/issues#6132
phiz71 added a commit to gravitee-io/gravitee-docker that referenced this issue Oct 27, 2021
* release both HTTP and kafka connectors
* but only add HTTP connector in APIM & GW bundles

gravitee-io/issues#6132
gcusnieux pushed a commit to gravitee-io/gravitee-api-management that referenced this issue Oct 27, 2021
Make the endpoints management extendable to allow others kind of endpoints like Kafka, MQTT, AMQP, …
Extract HTTP Endpoint & GRPC Endpoint management into a dedicated plugin.
Add connector-http in distribution pom.xml

gravitee-io/issues#6132
phiz71 added a commit to gravitee-io/gravitee-docker that referenced this issue Oct 27, 2021
* release both HTTP and kafka connectors
* but only add HTTP connector in APIM & GW bundles

gravitee-io/issues#6132
phiz71 added a commit to gravitee-io/gravitee-docker that referenced this issue Oct 27, 2021
* release both HTTP and kafka connectors
* but only add HTTP connector in APIM & GW bundles

gravitee-io/issues#6132
phiz71 added a commit to gravitee-io/gravitee-docker that referenced this issue Oct 27, 2021
* release both HTTP and kafka connectors
* but only add HTTP connector in APIM & GW bundles

gravitee-io/issues#6132
gcusnieux pushed a commit to gravitee-io/gravitee-api-management that referenced this issue Oct 27, 2021
Make the endpoints management extendable to allow others kind of endpoints like Kafka, MQTT, AMQP, …
Extract HTTP Endpoint & GRPC Endpoint management into a dedicated plugin.
Add connector-http in distribution pom.xml

gravitee-io/issues#6132
gcusnieux added a commit to gravitee-io/gravitee-definition that referenced this issue Oct 27, 2021
gcusnieux added a commit to gravitee-io/gravitee-connector-http that referenced this issue Oct 27, 2021
phiz71 pushed a commit to gravitee-io/gravitee-connector-http that referenced this issue Oct 27, 2021
phiz71 pushed a commit to gravitee-io/gravitee-definition that referenced this issue Oct 27, 2021
kamiiiel pushed a commit to gravitee-io/gravitee-node that referenced this issue Oct 27, 2021
phiz71 pushed a commit to gravitee-io/release that referenced this issue Oct 28, 2021
@LiliaEn
Copy link
Contributor

LiliaEn commented Nov 2, 2021

Please see bellow the scenarios that could not be properly tested and the questions.

  • Http protocol 1.1 vs 2.0 -> How can a call be blocked in order to check that the protocols are applied correctly?
  • Keep Alive-> How can a failed call be triggered to check this option properly?
  • Http Pipelining- > How can this option be tested properly?
  • Compression -> How can a compressed response be triggered?
  • HTTP Redirects -> Tried to applied Dynamic Routing policy to trigger the redirect, but I was getting 400. Is there another way to trigger 3xx status codes?
  • Http Headers-> Is there a header that can be used which will have an obvious outcome? Tried to use Connection or Content-type, but nothing special happens
  • Proxy-> How can proxy types be tested? Do we have some test data to check that?
  • SSL Options-> How can different key stores be tested?

cc: @RubenMMSantos @mouligno @gcusnieux

@gcusnieux
Copy link
Member

Please see bellow the scenarios that could not be properly tested and the questions.

* Http protocol 1.1 vs 2.0 -> How can a call be blocked in order to check that the protocols are applied correctly?

https://api.gravitee.io supports protocol 1.1 & 2.0. If both works, it's okay !

* HTTP Redirects -> Tried to applied Dynamic Routing policy to trigger the redirect, but I was getting 400. Is there another way to trigger 3xx status codes?

You can try to use another API with policy mock as endpoint and configure a 3xx status code

* Http Headers-> Is there a header that can be used which will have an obvious outcome? Tried to use Connection or Content-type, but nothing special happens

We can use https://api.gravitee.io/echo as endpoint and see headers in the response body

* Keep Alive-> How can a failed call be triggered to check this option properly?

It will not be possible to failed.

* Http Pipelining- > How can this option be tested properly?
* Compression -> How can a compressed response be triggered?
* Proxy-> How can proxy types be tested? Do we have some test data to check that?
* SSL Options-> How can different key stores be tested?

For all these options, the only way to verify is to monitor http calls between the gateway and the endpoint with a third party tool.
We have automatic integration tests for that, we can consider that it works correctly.

cc: @RubenMMSantos @mouligno @gcusnieux

@LiliaEn
Copy link
Contributor

LiliaEn commented Nov 15, 2021

Tested on the default environment version 3.13.0-SNAPSHOT.
Browser: Chrome Version 94.0.4606.81, macOS Big Sur v11.6
The following issues were reported: #6487, #6514 , #6538-> might be invalid (still under investigation)
The ticket will be blocked until the bugs will be fixed.

@LiliaEn
Copy link
Contributor

LiliaEn commented Nov 16, 2021

Above issues were fixed. Ticket is ready for release.

@phiz71 phiz71 changed the title [SME] API HTTP Connector Integration [management] Service Management Ecosystem (SME): API HTTP Connector Integration Nov 19, 2021
@phiz71 phiz71 closed this as completed Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants