Skip to content

Module's rest service is not executed in the context of the KB that i…#810

Merged
iroqueta merged 1 commit into
masterfrom
RestServicesInModules
Dec 8, 2023
Merged

Module's rest service is not executed in the context of the KB that i…#810
iroqueta merged 1 commit into
masterfrom
RestServicesInModules

Conversation

@iroqueta

@iroqueta iroqueta commented Dec 6, 2023

Copy link
Copy Markdown
Collaborator

…mports the module

Issue: 106023

@genexusbot

Copy link
Copy Markdown
Collaborator
Cherry pick to beta success

@tomas-sexenian tomas-sexenian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If possible, I think it would be better to use try-with-resources in those classes that implement the closable interface. For example XMLReader in AddExternalServiceFile and InputStream in SelectImport

@iroqueta iroqueta merged commit 17e05b8 into master Dec 8, 2023
@iroqueta iroqueta deleted the RestServicesInModules branch December 8, 2023 17:45
iroqueta added a commit that referenced this pull request Dec 29, 2023
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