allowLinking does not work

[tronicek]

Brief Summary

The application does not follow symbolic links when a property allowLinking in the virtual server is set to true.

Expected Outcome

Payara is expected to follow symbolic links in the application folder. It works in Eclipse Glassfish.

Current Outcome

Payara does not follow symbolic links in the application folder.

Reproducer

1. Deploy a web application.
2. Create a symbolic link in payara6/glassfish/domains/domain1/application/yourapp~v1. For example, cd payara6/glassfish/domains/domain1/application/yourapp~v1; ln -s ~/pdf.
3. In admin console, add property allowLinking in server-config -> Virtual Servers -> server and set its value to true.
4. Access files in the pdf folder, such as https://www.yourname.cz/yourapp/pdf/1.pdf.

Operating System

Amazon Linux

JDK Version

OpenJDK Runtime Environment Corretto-17.0.10.8.1

Payara Distribution

Payara Server Full Profile

[felixif]

Hello @tronicek,

I have tested the symbolic links, with the only difference being that I created the symbolic link this way: ln -s ~/Documents/payara-6.2024.2/payara6/glassfish/domains/domain1/application/ci6357-0.1-SNAPSHOT/pdf ~/pdf, followed by mkdir ~/Documents/payara-6.2024.2/payara6/glassfish/domains/domain1/application/ci6357-0.1-SNAPSHOT/pdf. Then I dropped one of our guides in the pdf folder, and I can access it via Payara Server with no issue on the /<appname>/pdf/1.pdf url

Your command created a symbolic link from the pdf folder inside "yourappname" pointing towards the home pdf folder, while my command does the opposite. Using the readlink CLI utility on the pdf folder inside "yourappname", my method prints home/felix/Documents/payara-6.2024.2/payara6/glassfish/domains/domain1/application/ci6357-0.1-SNAPSHOT/pdf/1.pdf, while your method prints /home/felix/pdf . I am not entirely sure if this is a bug or expected behaviour, so I will bring this issue up with my teammates. Meanwhile, the above method should work just fine. Please let me know if you managed to get the symbolic links working.

Best regards, Felix Ifrim

[tronicek]

Hi @felixif,

ln -s ~/Documents/payara-6.2024.2/payara6/glassfish/domains/domain1/application/ci6357-0.1-SNAPSHOT/pdf ~/pdf creates a link ~/pdf that points to ~/Documents/payara.../pdf. When you drop a file in ~/pdf, the file is created in ~/Documents/payara.../pdf, which is under your application root. You can check it simply by inspecting that folder. When you access that file, no symbolic link is followed and it is no surprise that it works.

To prevent misunderstanding, let me describe once more what I expect from allowLinking: I created a link in the application root (payara6/glassfish/domains/domain1/application/yourappv1) that pointed to a folder out of the application root and expected Payara to follow the link. It works under Glassfish but not under Payara. The link you create is always the same regardless of what command you use to create it because it is simply a file that contains the absolute path to the destination folder.

[github-actions[bot]]

Greetings, It's been more than 5 days since we requested more information or an update from you on the details of this issue. Could you provide an update soon, please? We're afraid that if we do not receive an update, we'll have to close this issue due to inactivity.

[tronicek]

I replied to your comments last week.

[aubi]

The allowLinking is still described in our documentation, so it should work. I tried it with clusterjsp as a reproducer and it really doesn't work. Using latest Payara 6.2024.2.

I tried link both directory and a file:

glassfish/domains/domain1/applications/clusterjsp$ ln -s /home/aubi/tmp
glassfish/domains/domain1/applications/clusterjsp$ ln -s /home/aubi/tmp/x.png

I set the allowLinking to true as described, in server-config -> Virtual Servers -> server, Additional Properties

No request succeeded, all return 404:

http://localhost:8080/clusterjsp/x.png
HTTP Status 404 - Not Found
http://localhost:8080/clusterjsp/tmp/x.png
HTTP Status 404 - Not Found

Note for myself: Something wrong with allowLinking in FileDirContext.validate()

@felixif Can we remove the Status: Abandoned? It's obviously wrong. Thanks.

[github-actions[bot]]

Greetings, It's been more than 5 days since this issue was identified as abandoned. We have closed this issue due to inactivity, please feel free to re-open it if you have more information to share.

[tronicek]

Are you serious?

[felixif]

Hello @tronicek,

Apologies for the issue being mislabeled as Abandoned and also for waiting so long for my answer. Me and Petr had a conversation where he explained my mistake in understanding symbolic links. I was able to reproduce the problem with no issues, so I will mark it as accepted. The issue has been raised to our Engineering team, codename FISH-8399.

Thank you very much for your report, and I apologise for the delay and the lack of understanding on my part.

Best regards, Felix

[tronicek]

Hi @felixif,

No problem. I am happy you are going to fix it because I really use this feature.