starred/dehydrated
1
0
Fork 0
mirror of https://github.com/dehydrated-io/dehydrated.git synced 2026-01-11 22:30:44 +01:00
Code Issues 79 Packages Projects Releases 10 Wiki Activity

script for hooks can not be found #62

New Issue
Closed
opened 2025-12-29 00:23:55 +01:00 by adam · 8 comments
adam commented 2025-12-29 00:23:55 +01:00
Owner
Copy Link

Originally created by @beelbrecht on GitHub (Feb 17, 2016).

Hi,

I'm using this config to specify the path of my hook.sh script:

HOOK="${BASEDIR}/hook.sh"

But my BASEDIR starts with /Volumes/Macintosh HD/Users
There is a space in the pathname. Why not adding double quotes to the hook variable ${HOOK} in letsencrypt.sh?
My shell script knowledge ist not very well, but this solution works for me...

Kind regards,
Benjamin

Originally created by @beelbrecht on GitHub (Feb 17, 2016). Hi, I'm using this config to specify the path of my hook.sh script: `HOOK="${BASEDIR}/hook.sh"` But my BASEDIR starts with `/Volumes/Macintosh HD/Users` There is a space in the pathname. Why not adding double quotes to the hook variable ${HOOK} in letsencrypt.sh? My shell script knowledge ist not very well, but this solution works for me... Kind regards, Benjamin
adam closed this issue 2025-12-29 00:23:55 +01:00
adam commented 2025-12-29 00:23:55 +01:00
Author
Owner
Copy Link

@lukas2511 commented on GitHub (Feb 17, 2016):

oh yea, spaces do exist, i always forget.

@lukas2511 commented on GitHub (Feb 17, 2016): oh yea, spaces do exist, i always forget.
adam commented 2025-12-29 00:23:56 +01:00
Author
Owner
Copy Link

@K2rool commented on GitHub (Mar 14, 2016):

Hi, this commit cusses hook script not to be loaded ./letsencrypt.sh: line 410: python hooks/cloudflare/hook.py: No such file or directory, i manage to get it working by undoing all the change this commit dose by removed all the double quotes which was added.

@K2rool commented on GitHub (Mar 14, 2016): Hi, this commit cusses hook script not to be loaded ./letsencrypt.sh: line 410: python hooks/cloudflare/hook.py: No such file or directory, i manage to get it working by undoing all the change this commit dose by removed all the double quotes which was added.
adam commented 2025-12-29 00:23:56 +01:00
Author
Owner
Copy Link

@lukas2511 commented on GitHub (Mar 14, 2016):

@K2rool just use hooks/cloudflare/hook.py, add a shebang to the first line of your script #!/usr/bin/env python and make it executable (chmod +x hooks/cloudflare/hook.py).

@lukas2511 commented on GitHub (Mar 14, 2016): @K2rool just use `hooks/cloudflare/hook.py`, add a shebang to the first line of your script `#!/usr/bin/env python` and make it executable (`chmod +x hooks/cloudflare/hook.py`).
adam commented 2025-12-29 00:23:56 +01:00
Author
Owner
Copy Link

@K2rool commented on GitHub (Mar 14, 2016):

@lukas2511 Thanks that fixed it, i got letsencrypt.sh on a few server i coped the config file from one of my other server which must be running a letsencrypt.sh version before this commit.

Thank for the quick reply and helping me fix my issue

@K2rool commented on GitHub (Mar 14, 2016): @lukas2511 Thanks that fixed it, i got letsencrypt.sh on a few server i coped the config file from one of my other server which must be running a letsencrypt.sh version before this commit. Thank for the quick reply and helping me fix my issue
adam commented 2025-12-29 00:23:56 +01:00
Author
Owner
Copy Link

@bviktor commented on GitHub (May 12, 2016):

Unfortunately this does not work for Java where you don't have a shebang.

./letsencrypt.sh: line 616: java -jar /opt/certzure/certzure.jar: No such file or directory

Any ideas how to solve this, other than creating a passthrough shell script? eval or subshell, maybe?

@bviktor commented on GitHub (May 12, 2016): Unfortunately this does not work for Java where you don't have a shebang. ``` ./letsencrypt.sh: line 616: java -jar /opt/certzure/certzure.jar: No such file or directory ``` Any ideas how to solve this, other than creating a passthrough shell script? eval or subshell, maybe?
adam commented 2025-12-29 00:23:57 +01:00
Author
Owner
Copy Link

@bviktor commented on GitHub (May 12, 2016):

The only other thing I can think of ATM is an additional, optional argument, "hook interpreter", that could contain "sh", "python2.7", "java -jar" or whatever. Opinions?

@bviktor commented on GitHub (May 12, 2016): The only other thing I can think of ATM is an additional, optional argument, "hook interpreter", that could contain "sh", "python2.7", "java -jar" or whatever. Opinions?
adam commented 2025-12-29 00:23:57 +01:00
Author
Owner
Copy Link

@lukas2511 commented on GitHub (May 13, 2016):

@bviktor I think a passthrough/wrapper script would be better than a new option. It's simple, it doesn't add much weight and it can handle all kind of edgecases.

@lukas2511 commented on GitHub (May 13, 2016): @bviktor I think a passthrough/wrapper script would be better than a new option. It's simple, it doesn't add much weight and it can handle all kind of edgecases.
adam commented 2025-12-29 00:23:58 +01:00
Author
Owner
Copy Link

@bviktor commented on GitHub (May 13, 2016):

Alright.

@bviktor commented on GitHub (May 13, 2016): Alright.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Complicated
bug
enhancement
help wanted
pull-request
Mirrored from GitHub Pull Request
 suggestions wanted
testing
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/dehydrated#62
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?