Skip to content

Conversation

@iroqueta
Copy link
Collaborator

@iroqueta iroqueta commented Mar 25, 2022

For any configuration Setting, this will be the order in which values are read from the Environment Variables:

  1. GX_COM_MYNAMESPACE_DEFAULT_DB_URL (with Namespace)
  2. GX__DEFAULT_DB_URL (without Namespace)

Issue: 95391

…ronment variables, first try to find it without namespace. For example “GX _DEFAULT_DB_URL” before GX_COM_MYNAMESPACE_DEFAULT_DB_URL”

Issue: 95391
@iroqueta iroqueta requested a review from ggallotti March 25, 2022 14:32
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

…ronment variables if dont exists try to find it without namespace. For example “GX _DEFAULT_DB_URL” after GX_COM_MYNAMESPACE_DEFAULT_DB_URL”

Issue: 95391
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@ggallotti ggallotti changed the title When try to read gx properties that are in datasource section in envi… Support Reading Configuration Settings without specifying the App Namespace within the KEY Mar 25, 2022
…ronment variables if dont exists try to find it without namespace. For example “GX _DEFAULT_DB_URL” after GX_COM_MYNAMESPACE_DEFAULT_DB_URL”

Issue: 95391
Copy link
Member

@ggallotti ggallotti left a comment

Choose a reason for hiding this comment

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

Seems "java/src/test/java/com/mockdb/GXcfg.java" was commited by accident in this PR.

{
public static int strcmp( String Left ,
String Right )
{
Copy link
Member

Choose a reason for hiding this comment

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

Was this file (mockdb/GXcfg.java) intended to be pushed in this commit? Seems that is from other PullRequest.

@genexusbot
Copy link
Collaborator

Cherry pick to beta success
  • 1f46060 (warning: no changes merged)

@iroqueta iroqueta merged commit 778f400 into master Jun 9, 2022
@iroqueta iroqueta deleted the ReadNamespceGXEnvVar branch June 9, 2022 18:56
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.

5 participants