-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Use the 'Location:' header only for latest FTL tag API check. #3987
Conversation
Signed-off-by: Dan Schaper <dan.schaper@pi-hole.net>
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pi-hole-update-wird-nicht-installiert/42862/11 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/ftl-keeps-updating-to-same-version/42898/2 |
Hi dschaper, which code do I use, I'm bit confuse about the codes look too complex. pls help afap thank you. |
Has this issue been fixed yet? |
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.
what does red and green colour indicate?
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pihole-c-causes-pihole-ftl-log-errors-ftl-v5-3-4/42154/28 |
By submitting this pull request, I confirm the following:
please fill any appropriate checkboxes, e.g: [X]
git rebase
)What does this PR aim to accomplish?:
Fixes #3986
GitHub API headers now have CSP with the work "location" in the policy. FTL tags came back with that location as the tag.
How does this PR accomplish the above?:
Only use
Locaition:
which should only grab the proper header.