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

Checking running status of OpenRefine with wget will not work correctly #6622

Closed
ostephens opened this issue May 22, 2024 · 0 comments · Fixed by #6626
Closed

Checking running status of OpenRefine with wget will not work correctly #6622

ostephens opened this issue May 22, 2024 · 0 comments · Fixed by #6626
Labels
Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. Type: Bug Issues related to software defects or unexpected behavior, which require resolution.
Milestone

Comments

@ostephens
Copy link
Sponsor Member

ostephens commented May 22, 2024

The PR #6361 which was meant to fix #2000 changes the check_running() code in the refine script. It introduces a variable URL_LOCAL="http://127.0.0.1" alongside the existing variable URL="http://${REFINE_HOST_INTERNAL}:${REFINE_PORT}/".

  • if cURL is used to check for a running OpenRefine instance, it checks using the URL variable
  • if wget is used to check for a running OpenRefine instance, it checks using the URL_LOCAL variable

This means wget will never fetch the OpenRefine page because it doesn't use the specified host or port that OpenRefine is/should be expected to be running on.

I believe that the lines for wget

no_proxy=127.0.0.1 wget -O - $URL_LOCAL > /dev/null 2>&1

RUNNING=`no_proxy=127.0.0.1 wget -O - $URL_LOCAL | grep "$CHECK_STR"`

should check the URL not URL_LOCAL and URL_LOCAL variable can be removed as it is redundant

@ostephens ostephens added Type: Bug Issues related to software defects or unexpected behavior, which require resolution. Status: Pending Review Indicates that the issue or pull request is awaiting review by project maintainers or collaborators labels May 22, 2024
@wetneb wetneb added Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. and removed Status: Pending Review Indicates that the issue or pull request is awaiting review by project maintainers or collaborators labels May 22, 2024
dino2580 added a commit to dino2580/OpenRefine that referenced this issue May 23, 2024
OpenRefine#6622

Removed use of URL_LOCAL and added URL in place of URL_LOCAL
@wetneb wetneb added this to the 3.8 milestone May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Indicates issues suitable for newcomers to design or coding, providing a gentle introduction. Type: Bug Issues related to software defects or unexpected behavior, which require resolution.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants