forked from pi-hole/pi-hole
-
Notifications
You must be signed in to change notification settings - Fork 2
sync: master to development #1
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
Open
github-actions
wants to merge
141
commits into
development
Choose a base branch
from
master
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
141 commits
Select commit
Hold shift + click to select a range
dad6247
Merge pull request #4347 from pi-hole/development
PromoFaux b7bba6a
Validate when adding not when removing
yubiuser bc21a71
Add option to not reload
yubiuser a872fab
Validate on removal as well
yubiuser 596689b
Validate host/domain of Local DNS records as well
yubiuser 7cdd887
Merge pull request #4384 from pi-hole/tweak/local_dns_cname
DL6ER 8713135
update tests: remove fedora 32, add fedora 34 (#4403)
bcambl b5e0f14
Merge pull request #4405 from pi-hole/development
PromoFaux c1eb35a
Add custom.list (Local DNS Records) to debug log
yubiuser c6da1a3
Fix documentation; add some missing zones
aviddiviner 0c4e1b5
Merge pull request #4417 from aviddiviner/fix-rfc-config-docs
yubiuser e733553
Remove unused code from webpage.sh
yubiuser 996f8ff
Recommend apt instead of apt-get if updating the package cache failed…
yubiuser ac4a975
Allow users to skip setting static IP adress (#4419)
yubiuser cedd1a2
unit test for umask problems in #3177 and #2730 (#3191)
pvogt09 cdd4d9e
Update the tests (#4427)
PromoFaux d85fee2
Merge pull request #4420 from pi-hole/clean/webpage.sh
DL6ER 996a2c7
Use a fixed list height for network interface selection
MichaIng b750b01
Merge pull request #4434 from MichaIng/patch-1
yubiuser bc8150a
Clean up bash script formatting
jbzdarkid 671fcaf
Merge pull request #4085 from jbzdarkid/patch-3
yubiuser 14a379d
Fix generated /etc/os-release file in OS check test
MichaIng 941f90d
Merge pull request #4443 from MichaIng/patch-1
yubiuser bfda52e
Fix number of invalid domains
yubiuser e485a7b
Some shellchecks in basic-install.sh (#4088)
ChillerDragon ba6d700
Fix `pihole -v` output if WebAdmin not installed (#4370)
subnut 647ba6e
Rename variables to improve comprehensibility
yubiuser 4c267f7
Merge pull request #4445 from pi-hole/fix/counting
DL6ER e243c56
Unblock adlist domain during gravity run in NODATA mode
yubiuser ae39e33
Use exec to run gravity script (#4449)
yubiuser b52a3a0
Add comment help text to list function
yubiuser 8d6ce78
Allow qr code iframe
yubiuser 2eff53b
Allow qr code iframe
yubiuser a3cc5df
Configure stale action (#4269)
dschaper 28085cf
Merge iFrame exceptions
yubiuser a780fc5
Set DBFile permissions on creation.
dschaper 76ae756
Check for DNS before run.
dschaper 533a77d
Add database function failure guards.
dschaper ef0a22f
Merge pull request #4478 from pi-hole/fix/db_permission
DL6ER c0f454d
Add new interface listening option "bind" (#4476)
DL6ER 3a67d1c
Merge pull request #4461 from pi-hole/qr_iframe
DL6ER 31de661
Merge pull request #4414 from pi-hole/debug/custom.list
DL6ER 669f1b0
Address reviewer's comment
yubiuser 190ab79
Implement fully-automated gravity database recovery method.
DL6ER 469c179
Return early from recovery routine when integrity checks didn't show …
DL6ER ab4bce4
Allow users to force recovery even when checks are okay using "pihole…
DL6ER b06efb6
Declare variables local
DL6ER 39a66b6
Replace Contributing Guide by link to docs.pi-hole.net (#4433)
yubiuser 7167e6d
Apply suggestions from code review
DL6ER db5e94b
use +short and omit obsolet awk
yubiuser 54c5832
Merge pull request #4450 from pi-hole/unblock_NODATA
DL6ER ff4487f
Escape quotes
yubiuser 1eb3117
Merge pull request #4455 from pi-hole/comment
DL6ER 920cf6d
Check for updates on master based on tags not commits (#4475)
yubiuser 2e1ce7f
Apply suggestions from code review
DL6ER d2a98ae
Document -r recover force case
DL6ER 3d3bb45
Merge pull request #4288 from pi-hole/new/gravity_repair
DL6ER 1b809e4
Merge pull request #4480 from pi-hole/development
PromoFaux ef30a85
Include port in status function
yubiuser 9f0e0db
Fix analyse ports
yubiuser 71ebd64
mend
yubiuser 5ef731f
Fix indention
yubiuser 4a2f4c1
Fix indention_2
yubiuser 2a86941
Add netcat to dependencies
yubiuser 5729f64
Fix missing fi
yubiuser f45248d
Use FTL's new dns-port API endpoint
yubiuser 71ed842
Fixed path to 404 file when using custom.php (#4488)
Computroniks 2b52f92
Inlcude port also in cli output
yubiuser bcb5915
Analyse port also on ports other than 53
yubiuser 3989cc1
Remove double text output
yubiuser 886f0c7
Merge pull request #4485 from pi-hole/tweak/web_status
DL6ER 5bd7cc9
Replace `which` with `command -v` (#4499)
WaLLy3K 0e359a6
Set dnsmasq interface listening by default to local (#4509)
yubiuser d605b4b
Merge pull request #4513 from pi-hole/master
PromoFaux 241e53e
Skip debug upload question if called from web interface (#4494)
yubiuser c6a2a6f
Install pihole-FTL.conf template on fresh installation (#4496)
yubiuser 7807a93
If PIHOLE_DOCKER_TAG is set, then include that info in the debug run …
PromoFaux 5823f5e
Use ss instead of lsof (#4518)
DL6ER 2ff10fc
Merge pull request #4514 from pi-hole/development
PromoFaux 81698ef
Fix Pi-hole status in chronometer
yubiuser 0124e49
Merge pull request #4521 from pi-hole/fix/chronometer
PromoFaux 875ad04
Merge pull request #4522 from pi-hole/development
PromoFaux c208032
Install netcat-openbsd as dependency explicitly
MichaIng bfd9fe8
Remove debug from Stale
dschaper 363e2f1
Merge pull request #4534 from pi-hole/meta/enable_stale
PromoFaux 3097c8f
Skip the required ports check if installed in docker container. Unpri…
PromoFaux ce86157
Fix gravity in case there are no adlists at all or all are disabled (…
DL6ER 7010ed4
Merge pull request #4532 from MichaIng/patch-1
PromoFaux 1b0b24d
Merge pull request #4539 from pi-hole/master
PromoFaux a65a841
Remove oneline from ss call
yubiuser d5253f2
Merge pull request #4542 from pi-hole/remove_oneline
DL6ER b20b38d
Include ip addr show and ip route show for us to help with local-serv…
DL6ER ec3a5c2
Merge pull request #4543 from pi-hole/tweak/debug_ipaddr
DL6ER f124568
Add action to automatically sync master to dev when code is pushed to…
PromoFaux 65a0424
Merge pull request #4548 from pi-hole/actions/sync-to-dev
PromoFaux a79c115
Merge pull request #4550 from pi-hole/master
PromoFaux 3260cb4
ops per run -> 300 for stale
PromoFaux 918f7a5
Merge pull request #4554 from pi-hole/master
PromoFaux ed6b852
use sed substitute instead of delete and append (#4555)
lschloetterer 57ba60c
Change the exemption issue label pinned to internal for stale issues
yubiuser cdde832
Some use uppercase some don't...
yubiuser 6ead24b
Move space into variable (#4562)
rdwebdesign 3cd662e
Merge pull request #4558 from pi-hole/stale
PromoFaux e80a773
Merge pull request #4568 from pi-hole/master
PromoFaux 7aa28e4
Do a full fetch
yubiuser bad6d8a
add parameter to set filename for teleporter
lschloetterer f0f5cc5
Use internal SQLite3 engine in more places in gravity.sh
DL6ER 534f9a6
Copy database tables earlier into the new gravity database to avoid f…
DL6ER 5bb79de
Clean possible leftovers in domainlist_by_group, adlist_by_group, and…
DL6ER 8cbffa1
Replace remaining sqlite3 calls by calls to our embedded pihole-FTL s…
DL6ER 1dd9d55
Replace value for BLOCKING_ENABLED (and QUERY_LOGGING, for consistenc…
PromoFaux 2f4c4d9
Fix stale label to stale
yubiuser 74d7d10
Orphans need to be deleted in the old database
DL6ER 77e5121
Split new function out into a separte utility script and add a test f…
PromoFaux 5d68dac
Merge pull request #4588 from pi-hole/stale
PromoFaux 30ec1c9
Merge pull request #4593 from pi-hole/master
DL6ER e09dd56
Remove RPM package sqlite as well
DL6ER d956498
Merge pull request #4575 from pi-hole/fix/tag_update
DL6ER 76d4e12
Merge pull request #4585 from pi-hole/tweak/sed-add-if-not-exists
DL6ER 881d926
add hint for custom teleporter filename to help function
lschloetterer 844c4dc
Merge pull request #4584 from pi-hole/fix/gravity_internal_sqlite3
DL6ER 444526a
Switch from centos8 to centos8:stream base image for centos 8 tests
PromoFaux ee9f485
Merge pull request #4596 from pi-hole/long-live-centos8-stream
PromoFaux 7c60ee8
Remove pihole-FTL.conf man page
yubiuser d7d8e97
Remove pihole-FTL.conf.5 from automated tests
yubiuser c3c5342
Fix reviewer's comment
yubiuser 2a0bb5b
Create second entry for teleporter and adjust spacing
lschloetterer f342b2c
Merge pull request #4489 from pi-hole/tweak/manpages
PromoFaux 2bd670a
Merge pull request #4582 from lschloetterer/patch-1
PromoFaux 08999bf
Use case insensitive deletion when removing custom CNAME/DNS records …
PromoFaux e9250d6
Merge pull request #4598 from pi-hole/alt-4597
PromoFaux 6ffa2ba
Merge pull request #4547 from pi-hole/development
PromoFaux 9be5199
remove the CONTENT_COMPARISON setting (defaults to false)
PromoFaux 899cac0
Ignore Documentation Needed label
PromoFaux 2f38452
Wrap touch calls with if/then guards for Buster docker.
dschaper c2384ec
Change touch that would always fire to install.
dschaper d45c9fc
Final touch to install fix.
dschaper 8a5c7de
Ensure existing files are proper owner and mode.
dschaper 0f19299
Create empty files.
dschaper b714c45
Found it.
dschaper 326cd6a
Merge pull request #4665 from pi-hole/fix/touch_guard
dschaper File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| changelog: | ||
| exclude: | ||
| labels: | ||
| - internal | ||
| authors: | ||
| - dependabot | ||
| - github-actions |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| name: Mark stale issues | ||
|
|
||
| on: | ||
| schedule: | ||
| - cron: '0 * * * *' | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| stale: | ||
|
|
||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| issues: write | ||
|
|
||
| steps: | ||
| - uses: actions/stale@v4 | ||
| with: | ||
| repo-token: ${{ secrets.GITHUB_TOKEN }} | ||
| days-before-stale: 30 | ||
| days-before-close: 5 | ||
| stale-issue-message: 'This issue is stale because it has been open 30 days with no activity. Please comment or update this issue or it will be closed in 5 days.' | ||
| stale-issue-label: 'stale' | ||
| exempt-issue-labels: 'Internal, Fixed in next release, Bug: Confirmed, Documentation Needed' | ||
| exempt-all-issue-assignees: true | ||
| operations-per-run: 300 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| name: Sync Back to Development | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - master | ||
|
|
||
| jobs: | ||
| sync-branches: | ||
| runs-on: ubuntu-latest | ||
| name: Syncing branches | ||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v2 | ||
| - name: Opening pull request | ||
| id: pull | ||
| uses: tretuna/sync-branches@1.4.0 | ||
| with: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| FROM_BRANCH: 'master' | ||
| TO_BRANCH: 'development' | ||
| - name: Label the pull request to ignore for release note generation | ||
| uses: actions-ecosystem/action-add-labels@v1 | ||
| with: | ||
| labels: internal | ||
| repo: ${{ github.repository }} | ||
| number: ${{ steps.pull.outputs.PULL_REQUEST_NUMBER }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,3 +9,4 @@ __pycache__ | |
| *.egg-info | ||
| .idea/ | ||
| *.iml | ||
| .vscode/ | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Copilot Autofix
AI 4 months ago
To fix this problem, we should explicitly declare a
permissionsblock. The best place is at the job level (under thesync-branchesjob in this workflow), defining only the permissions required. Since this workflow checks out code and creates pull requests, and labels PRs, the minimal required permissions arecontents: read(for checkout) andpull-requests: write(to create and modify PRs and apply labels). No other write permissions appear necessary. Thepermissionsblock should be added as the first line inside the job definition, aboveruns-on. No special imports, definitions, or method changes are required. Only the YAML needs to be updated.