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

configuration parameter @web.application_root_url@ neglected in content view #438

Open
jane-heller-bkg opened this issue Jun 19, 2024 · 4 comments
Assignees
Labels
question Further information is requested under analysis A first analysis is performing

Comments

@jane-heller-bkg
Copy link

When viewing register items through the content view, the link presented in the column Status for each item is constructed falsely. It should probably follow @web.application_root_url@. Currently it is built using the local listen interface of the tomcat instance (e.g. http://127.0.0.1:8080//status/valid) and contains a duplicate slash.

@jane-heller-bkg
Copy link
Author

This is also related to #367
URL generation should probably be overhauled in generally, because it seems faulty at different places.

@iratigarzon
Copy link
Contributor

Dear @jane-heller-bkg,

The URL for the status column is based on the baseuri attribute of the reg_statusgroup table. If you encounter a duplicate slash in the URL, it may be because the baseuri attribute in reg_statusgroup is empty. Here is an example from our local instance to help you make a modification in the mentioned table.

In our case, we have the following information in the reg_statusgroup table:

image

When we click on the status column, it redirects us to http://host.docker.internal/registry/status/valid, showing the following information:

image

I hope this information helps you resolve the issue.

Best regards,

Irati.

@arantzaetxebarria arantzaetxebarria added question Further information is requested under analysis A first analysis is performing labels Jul 3, 2024
@arantzaetxebarria
Copy link
Collaborator

Dear @jane-heller-bkg

Did you find the above explanation provided useful?
Do you happen to have any news on this?

Best regards

@jane-heller-bkg
Copy link
Author

Hello @arantzaetxebarria

no, unfortunately updating the mentioned field in the database did not change the URL in frontend. (I issued: UPDATE reg_statusgroup SET baseuri = '{CORRECTURI}';

Also having multiple random places like different configuration variables, static variables in the code and database fields that are not documented and all should point to the same URL is in no way feasible. See also the other tickets: #367 and #438

Please update the software so that it generates URLs in a congruent manner, eg. fed from a single configuration variable.

Best

Jane

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested under analysis A first analysis is performing
Projects
None yet
Development

No branches or pull requests

3 participants