apinf / platform

Apinf - Open source API management platform with multi proxy and protocol support
https://apinf.com/
European Union Public License 1.1
74 stars 33 forks source link

Proxy path saving failure #3286

Closed preriasusi closed 6 years ago

preriasusi commented 6 years ago

Steps to reproduce:

  1. Go to API proxy settings
  2. Modify 'API base path'
  3. Push 'Save&Puplish' button:

Expected result:

'Api base path' get's updated - Tested by caling API from API url.

Real result:

'API base path' does not change on proxy.

Note: API base path gets updated only when 'Proxy base path' is modified also.

deligence-dharmendra commented 6 years ago

Hi @preriasusi @matleppa I have tested this functionality on my local system and its working fine. I have used 3 browsers to test the functionality. See test environment details.

Ubuntu 14.04.5 LTS,
Mozilla Firefox 57.0.1,
Google Chrome 60.0.3112.78
chromium-browser 63.0.3239.84,

Windows 7 64 bit
Internet Explorer 11.0.9600
Firefox 55.0.3
Google Chrome 63.0.32
Opera 49.0.27

Second, I have a question related to below code. I'm not familiar with Meteor.throw().

apinf/platform/blob/develop/apinf_packages/proxy_backends/client/form/autoform.js
        if (publishApiBackendOnApiUmbrellaError) {
           Meteor.throw(500, error);

Third, what is the use of below response.result? I'm getting the huge amount of data in this response.

apinf/platform/blob/develop/apinf_packages/proxies/server/methods/api_umbrella.js
try {
      // Send the API Backend to API Umbrella's endpoint for creation in the backend
      response.result = umbrella.adminApi.v1.apiBackends.createApiBackend(backend);

Please advise

preriasusi commented 6 years ago

@deligence-dharmendra Test by calling hte API. The UI updates, but not the real path to make API calls.

preriasusi commented 6 years ago

I can't repro anymore on 0.52.0 in production.