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]: PAVICS_FQDN_PUBLIC should be used in many places instead of PAVICS_FQDN #434

Open
mishaschwartz opened this issue Mar 7, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@mishaschwartz
Copy link
Collaborator

mishaschwartz commented Mar 7, 2024

Summary

In several locations across the code-base, the PAVICS_FQDN environment variable is used to display URLs that are user-facing. These should almost exclusively use the PAVICS_FQDN_PUBLIC variable instead.

⚠️
If #428 has been merged already, then the relevant variables have been changed to BIRDHOUSE_FQDN and BIRDHOUSE_FQDN_PUBLIC
⚠️

Details

The following are examples that have been flagged, there may be others as well and a full review of the code-base is needed where PAVICS_FQDN is used

magpie.url = https://${PAVICS_FQDN}/magpie

proxy_pass https://${PAVICS_FQDN}${TWITCHER_PROTECTED_PATH}/thredds/;

twitcher.url = https://${PAVICS_FQDN}/twitcher

magpie.url = https://${PAVICS_FQDN}/magpie

# Homepage URL: https://${PAVICS_FQDN}
# Authorization callback URL: https://${PAVICS_FQDN}/magpie/providers/github/signin

#export RAVEN_GEO_URL="https://${PAVICS_FQDN}/geoserver/"

So that the url ``https://<PAVICS_FQDN>/canarie/node/service/stats`` also return

proxy_pass http://${BIRDHOUSE_FQDN}${TWITCHER_PROTECTED_PATH}/geoserver-secured/;

curl --include --silent "https://${PAVICS_FQDN}/canarie/node/service/stats" | head

curl --write-out '%{http_code}' --output /dev/null --silent "https://${PAVICS_FQDN}/canarie/node/service/stats" \

curl --include --silent "https://${PAVICS_FQDN}/canarie/node/service/stats" | head

MAGPIE_SERVER_URL="https://$PAVICS_FQDN/magpie"

Environment

Information Value
Server/Platform URL all
Version Tag/Commit 1.25.6+
Related issues/PR
Related components
Custom configuration

Concerned Organizations

@mishaschwartz mishaschwartz added the bug Something isn't working label Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants