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

Gx web object refactoring #717

Merged
merged 5 commits into from
May 17, 2023
Merged

Gx web object refactoring #717

merged 5 commits into from
May 17, 2023

Conversation

iroqueta
Copy link
Collaborator

No description provided.

iroqueta added 2 commits May 12, 2023 10:42
This allow run non FrontEnd object without Frontend support classes.
Issue:102619
This allow run non FrontEnd object without Frontend support classes.
Issue:102619
@iroqueta iroqueta requested a review from ggallotti May 12, 2023 13:45
@genexusbot
Copy link
Collaborator

Cherry pick to beta failed, 2 conflicted files in commit 416c3b5
  • java/src/main/java/com/genexus/GxRestService.java
  • java/src/main/java/com/genexus/internet/HttpContext.java

@@ -1537,6 +1453,13 @@
}
}

public void writeText(String text)
{
if (getResponseCommited())

Check failure

Code scanning / Fortify on Demand

Unreleased Resource: Streams

The function setStream() in HttpContextWeb.java sometimes fails to release a system resource allocated by getOutputStream() on line 1458.The program can potentially fail to release a system resource.
@iroqueta iroqueta removed the conflict label May 12, 2023
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

ggallotti
ggallotti previously approved these changes May 15, 2023
@ggallotti
Copy link
Member

(Theres seems to be a Fortify Issue.)

This allow run non FrontEnd object without Frontend support classes.
Issue:102619
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@iroqueta iroqueta merged commit c9ca6e6 into master May 17, 2023
@iroqueta iroqueta deleted the GXWebObjectRefactoring branch May 17, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants