Skip to content

Improvement/web api framework optimization #17

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.

Sign up for GitHub

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

Open
wants to merge 10 commits into
base: production/beta
Choose a base branch
from
Open

Improvement/web api framework optimization #17

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
107477b
Improved label finding
NielsdeBoerDAE Nov 4, 2025
750317e
Implemented sql constrains
NielsdeBoerDAE Nov 4, 2025
2746f4b
Turn find index for cRestChildCollection into a property
NielsdeBoerDAE Nov 4, 2025
568cecf
Allow query params to be passed twice
NielsdeBoerDAE Nov 4, 2025
2098ccd
Optimized json handles
NielsdeBoerDAE Nov 6, 2025
8e59a3a
Implement number field for openapi specification
NielsdeBoerDAE Nov 18, 2025
3dd3464
Log errors to event log
NielsdeBoerDAE Nov 18, 2025
2d21297
Reverse proxy support for servers
NielsdeBoerDAE Nov 18, 2025
24de861
Implement pagination
NielsdeBoerDAE Nov 18, 2025
2630ec5
GET requests should show that they return an array and not a single o…
NielsdeBoerDAE Nov 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 17 additions & 6 deletions Web API Library/AppSrc/WebApi/cOpenApiSpecification.pkg
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,9 @@ Class cOpenApiSpecification is a cObject
{ Visibility = Private }
Procedure GenerateServersInfo Handle hoOpenApiSpecJson
Handle hoServersJson hoServerJson hoHttpApi
String sApiPath sRouterPath sDescription sApiRoot sServerName sSecureString
String sApiPath sRouterPath sDescription sApiRoot sServerName sSecureString sForwardedProto
Boolean bSecure
Integer iServerPort

Get GetWebApiObject to hoHttpApi
Get psPath of hoHttpApi to sApiPath
Expand All @@ -77,15 +78,25 @@ Class cOpenApiSpecification is a cObject
If (sApiRoot = "") Begin
Get ServerVariable of ghoWebServiceDispatcher "SERVER_NAME" to sServerName
Get ServerVariable of ghoWebServiceDispatcher "URL" to sApiRoot
Copy link
Owner

@DataFlex-dev DataFlex-dev Nov 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the future see if we can check a forwarded for server name and url as well.

Get ServerVariable of ghoWebServiceDispatcher "SERVER_PORT_SECURE" to bSecure
Get ServerVariable of ghoWebServiceDispatcher "HTTP_X_FORWARDED_PROTO" to sForwardedProto

//Add http or https based on the SERVER_PORT_SECURE setting
If bSecure Begin
Move "https://" to sSecureString
// Check for reverse proxy
If (sForwardedProto <> "") Begin
Move (sForwardedProto + "://") to sSecureString
End
Else Begin
Move "http://" to sSecureString
Get ServerVariable of ghoWebServiceDispatcher "SERVER_PORT" to iServerPort
// 443 is the default secure port
Move (iServerPort = 443) to bSecure
//Add http or https based on the SERVER_PORT setting
If bSecure Begin
Move "https://" to sSecureString
End
Else Begin
Move "http://" to sSecureString
End
End

//The url contains /OpenApi since its part of the requesting url, we can take it out of the url.
Move (Replace("/OpenApi", sApiRoot, "")) to sApiRoot
Move (sSecureString + sServerName + sApiRoot) to sApiRoot
Expand Down