-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add OpenBSD rc.d script. #736
base: main
Are you sure you want to change the base?
Conversation
@favonia I tried to keep the style of your README, feel free to edit if you want it laid out differently. I also named the directory in the repo as |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #736 +/- ##
=======================================
Coverage 95.67% 95.67%
=======================================
Files 58 58
Lines 2635 2635
=======================================
Hits 2521 2521
Misses 104 104
Partials 10 10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This has a few prerequisites: - Creating a `_cloudflare_ddns` user - Add a `cloudflare_ddns` section to your `/etc/login.conf` (for env variables, this is noted in the accompanying `README` commit Relies on favonia#733
0a24e48
to
65448ad
Compare
@skarekrow Thank you! Could you put the files under |
@favonia Whoops didn't even see the |
_(contributed by [Brandon @skarekrow](https://github.com/skarekrow))_ | ||
|
||
To use: | ||
- Simply copy the [rc.d](./openbsd/cloudflare_ddns) script located in the repo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Simply copy the [rc.d](./openbsd/cloudflare_ddns) script located in the repo | |
- Copy the shipped [rc.d script](./openbsd/cloudflare_ddns) into `/etc/rc.d/` |
|
||
```sh | ||
cloudflare_ddns:\ | ||
:setenv=CF_API_TOKEN=YOUR_TOKEN,DOMAINS=THE_DOMAINS_YOU_WISH_TO_USE,EMOJI=false:\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:setenv=CF_API_TOKEN=YOUR_TOKEN,DOMAINS=THE_DOMAINS_YOU_WISH_TO_USE,EMOJI=false:\ | |
:setenv=EMOJI=false,CF_API_TOKEN=YOUR-CLOUDFLARE-API-TOKEN,DOMAINS=YOUR-DOMAINS:\ |
@skarekrow I tweaked this line because
- It looks easy to accidentally change
EMOJI=false
when editing the domains if it goes to the end. - The updater will print better error messages for the special value "YOUR-CLOUDFLARE-API-TOKEN".
However I have one question---the domains are comma-separated, but it seems setenv
is also comma-separated. Wouldn't that be an issue? 🤔
This has a few prerequisites:
_cloudflare_ddns
usercloudflare_ddns
section to your/etc/login.conf
(for env variables, this is noted in the accompanyingREADME
commitRelies on #733