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

Bug Report: System property placeholder interpolation does not work if property with placeholder comes first in domain.xml/FISH-5855 #5170

Open
kruemelmonstr opened this issue Mar 21, 2021 · 5 comments
Assignees
Labels
Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect

Comments

@kruemelmonstr
Copy link

Description


Hey Payara Support,

I am currently facing a very strange behaviour: imagine you have two system properties in your domain.xml like this:

<system-property name="property.name.one" value="${property.name.two} one"></system-property>
<system-property name="property.name.two" value="two"></system-property>

After the deployment property "one"s placeholder is not replaced with the value from property two. If change the order of the properties manually and restart the server everything works as expected. A normal restart with order change does not has an impact.

Since I create the whole domain configuration with asadmin command if no direct influence in what order the properties are stored in domain.xml file.

Expected Outcome

The order of the system properties should not have any impact on the placeholder interpolation.

Current Outcome

Placeholder is not replaced in property string

Steps to reproduce

Environment

  • Distribution:
    Server Full Profile - Payara Server 4.1.2.191 #badassfish (build 872)
  • JDK Version:
    ZuluJDK 8u282
  • Operating System:
    Linux
@kruemelmonstr kruemelmonstr added the Type: Bug Label issue as a bug defect label Mar 21, 2021
@AlanRoth AlanRoth added the Status: Open Issue has been triaged by the front-line engineers and is being worked on verification label Mar 22, 2021
@AlanRoth AlanRoth self-assigned this Mar 22, 2021
@AlanRoth
Copy link

Hi, thanks for raising the issue,

Are you able to reproduce it in the latest Payara Community version? Can you also provide a self-contained reproducer with the steps you take to cause this bug?

Thank you,
Alan

@AlanRoth AlanRoth added Status: Pending Waiting on the issue requester to give more details or share a reproducer and removed Status: Open Issue has been triaged by the front-line engineers and is being worked on verification labels Mar 22, 2021
@kruemelmonstr
Copy link
Author

Hey @AlanRoth
I've created you a short example for you to reproduce the the problem.

https://github.com/kruemelmonstr/payara-property-replacement-bug
Hope that helps :)

Regards, Fabian

@AlanRoth AlanRoth added Status: Open Issue has been triaged by the front-line engineers and is being worked on verification and removed Status: Pending Waiting on the issue requester to give more details or share a reproducer labels Mar 23, 2021
@AlanRoth
Copy link

AlanRoth commented Jun 7, 2021

Hi Fabian,

Thank you for the reproducer, however you are using version 4.1.2.181 - if you require support for older versions in maintenance then I can refer you to the sales team to discuss a license for Enterprise or the inquire on the Payara Forum.

Are you able to provide a reproducer using the latest Payara Community version 5.2021.4? You may find that the bug has already been fixed in recent versions.

Thank you for your understanding,
Alan

@AlanRoth AlanRoth added Status: Pending Waiting on the issue requester to give more details or share a reproducer and removed Status: Open Issue has been triaged by the front-line engineers and is being worked on verification labels Jun 7, 2021
@kruemelmonstr
Copy link
Author

Hey @AlanRoth,

Sorry for the delayed response - I havn't seen your reply. Unfortunately I was also able to reproduced this behaviour successfully with the latest community version 5.2021.4 as requested. So the defect has not been fixed completly.

See the updated version kruemelmonstr/payara-property-replacement-bug@74ae76e

Regards,
Fabian

@AlanRoth AlanRoth added Status: Open Issue has been triaged by the front-line engineers and is being worked on verification and removed Status: Pending Waiting on the issue requester to give more details or share a reproducer labels Jul 21, 2021
@fturizo fturizo assigned shub8968 and unassigned AlanRoth Nov 9, 2021
@shub8968
Copy link
Contributor

Hi @kruemelmonstr,

I am able to reproduce this issue. I have raised an internal issue with JIRA FISH-5855. Thanks. It may take a while to fix the issue. You are free to submit a PR to fix this issue if you feel confident to do so.

@shub8968 shub8968 added Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev and removed Status: Open Issue has been triaged by the front-line engineers and is being worked on verification labels Nov 12, 2021
@shub8968 shub8968 changed the title Bug Report: System property placeholder interpolation does not work if property with placeholder comes first in domain.xml Bug Report: System property placeholder interpolation does not work if property with placeholder comes first in domain.xml/FISH-5855 Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect
Projects
None yet
Development

No branches or pull requests

3 participants