Navigation Menu

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

Failure to download FTL binary should stop script. #2324

Closed
3 tasks done
dschaper opened this issue Aug 6, 2018 · 2 comments
Closed
3 tasks done

Failure to download FTL binary should stop script. #2324

dschaper opened this issue Aug 6, 2018 · 2 comments
Assignees
Labels
Bug: Confirmed Verified as a bug to be resolved Discussion Fixed in next release triage: Issue User Filled out Issue Template
Milestone

Comments

@dschaper
Copy link
Member

dschaper commented Aug 6, 2018

In raising this issue, I confirm the following: {please fill the checkboxes, e.g: [X]}

How familiar are you with the the source code relevant to this issue?:

10

Expected behaviour:

A failure to get the FTL binary should be a dependency that is checked before the bulk of the script runs. This will allow for a failure of the script and no modification of the target system. Will allow re-runs of the script to be easier, no need to have to edit /etc/resolv.conf to change the localhost entry to an upstream.

Actual behaviour:

FTL failures happen far in to the process after dependencies are installed and resolv.conf has already been modified to 127.0.0.1.

Steps to reproduce:

Just run the installer or try to update.

Debug token provided by uploading pihole -d log:

N/A

@dschaper dschaper added Bug: Confirmed Verified as a bug to be resolved Discussion triage: Issue User Filled out Issue Template Investigating Open issue or bug, under investigation labels Aug 6, 2018
@dschaper dschaper added this to the 4.1 milestone Aug 6, 2018
@dschaper dschaper self-assigned this Aug 6, 2018
@AzureMarker AzureMarker modified the milestones: 4.1, v4.2 Dec 7, 2018
@AzureMarker
Copy link
Contributor

Pushed back to v4.2 so we can get v4.1 out soon.

@dschaper dschaper added Fixed in next release and removed Investigating Open issue or bug, under investigation labels Feb 3, 2019
@dschaper
Copy link
Member Author

dschaper commented Feb 3, 2019

Fixed in #2600

@dschaper dschaper closed this as completed Feb 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Confirmed Verified as a bug to be resolved Discussion Fixed in next release triage: Issue User Filled out Issue Template
Projects
None yet
Development

No branches or pull requests

2 participants