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

FIWARE integration task#1 #3552

Closed ilarimikkonen closed 6 years ago

ilarimikkonen commented 6 years ago

APInf needs to expose two settings from API umbrella:

see screencaps

ilarimikkonen commented 6 years ago

task1.PNG task2.PNG task3.PNG

ilarimikkonen commented 6 years ago

screencaps are for illustration; changes via management API

ilarimikkonen commented 6 years ago
preriasusi commented 6 years ago

3552

preriasusi commented 6 years ago

Regex to be .* when writing Required Headers.

preriasusi commented 6 years ago

UI_3551.png

preriasusi commented 6 years ago

UI design there. Should it be grouped still under a "OAuth IDP settings" etc.?

fdelavega commented 6 years ago

The required headers stuff under the sub-url parameters is used to specify tenants of the different datasets which are registered using the sub-url settings.

In this regard, if the required headers functionality is going to be supported, then the method and URL regex for sub-urls need to be also supported

matleppa commented 6 years ago

@fdelavega About parameter Required Headers (RH).

ilarimikkonen commented 6 years ago

confirmed with Ficodes: we need to support all methods including "any". Supporting multiple rules for same method is optional. regex needs to be given separately to all methods once / method. Each setting group has three parameters, method (pulldown), regex (text field) and required headers (text field)

ilarimikkonen commented 6 years ago

ficodeintegrationUi.png

please use this style of ui implementation, this is visible for example in nightly if you select "custom rate limits" in proxy settings

ficodeintegrationUi2.PNG

ilarimikkonen commented 6 years ago

setting can be placed after rate limits in the proxy setting UI. For initial implementation it can be visible all the time

ilarimikkonen commented 6 years ago

there is one "IDP App ID" per API

Nazarah commented 6 years ago

@vinaytech please add me as a reviewer when u create a PR for this issue. I'd like to check the UI.