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

Bugfix: wrong arguments for requests_session #1719

Merged
merged 9 commits into from
May 18, 2022

Conversation

k9ert
Copy link
Contributor

@k9ert k9ert commented May 13, 2022

You cannot call it with the explorer as argument. Instead, we're now checking within the UtxoScanner for inconsitent arguments.

@netlify
Copy link

netlify bot commented May 13, 2022

Deploy Preview for specter-desktop-docs ready!

Name Link
🔨 Latest commit d6c0e65
🔍 Latest deploy log https://app.netlify.com/sites/specter-desktop-docs/deploys/6284b6083139a40009a8a44e
😎 Deploy Preview https://deploy-preview-1719--specter-desktop-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@k9ert
Copy link
Contributor Author

k9ert commented May 16, 2022

Two things are still missing in this PR:

  • A user choosing a tor-onion-address for the explorer should get a tor-capable session
  • A user who does the above but doesn't have a reasonable tor-setup, needs to get a reasonable error-message

@k9ert
Copy link
Contributor Author

k9ert commented May 16, 2022

Two things are still missing in this PR:

  • A user choosing a tor-onion-address for the explorer should get a tor-capable session
  • A user who does the above but doesn't have a reasonable tor-setup, needs to get a reasonable error-message

@k9ert
Copy link
Contributor Author

k9ert commented May 18, 2022

The second one was mitigated by reasonable error-logging.

@k9ert k9ert merged commit 9b01549 into cryptoadvance:master May 18, 2022
ankur12-1610 pushed a commit to ankur12-1610/specter-desktop that referenced this pull request May 18, 2022
* fix confused utxorescan

* switch arguments

* Yet another failure mode

* using the param as well

* Fix abortion of UTXO rescan

* forgotten commit

* onion-explorer support
ankur12-1610 pushed a commit to ankur12-1610/specter-desktop that referenced this pull request Jun 2, 2022
* fix confused utxorescan

* switch arguments

* Yet another failure mode

* using the param as well

* Fix abortion of UTXO rescan

* forgotten commit

* onion-explorer support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant