chore: update diagnostic-tools.md #1934
Merged
Conversation
This file contains 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
lidel
commented
Oct 2, 2024
|
|
||
| @[youtube](XeNOQDOrdC0) | ||
|
|
||
| Each error type output by the tool can indicate a solution to your problem: | ||
|
|
||
| - **Could not connect to the multiaddr** indicates that machines on the internet cannot talk to your machine. Fix your firewall, add port forwarding, or use a relay. | ||
| - **Could not find address in the DHT** indicates that your machine is either not connected to the Amino DHT (even as a client), or it is not advertising the address that you are using to test. | ||
| - **Multihash not advertised in the DHT** indicates that your machine has not advertised that it has the requested content in the Amino DHT. If you're using Kubo, enable the [Accelerated DHT Client](https://github.com/ipfs/go-ipfs/blob/master/docs/experimental-features.md#accelerated-dht-client). | ||
| - **Multihash not advertised in the DHT** indicates that your machine has not advertised that it has the requested content in the Amino DHT. |
There was a problem hiding this comment.
Rationale: we know there could be other reasons, such as people just unknowingly disabling announcements via lowpower profile (ipfs/kubo#10524).
Let's remove this kubo-specific suggestion, as it may do more harm than good (we know that it makes Kubo-based gateway perform worse with accelerated client enabled (notes))
lidel
commented
Oct 2, 2024
Comment on lines
-14
to
-17
| - [Project Apollo archives](https://explore.ipld.io/#/explore/QmSnuWmxptJZdLJpKRarxBMS2Ju2oANVrgbr2xWbie9b2D) | ||
| - [IGIS Git repository](https://explore.ipld.io/#/explore/baf4bcfg4ep767tjp5lxyanx5urpjjgx5q2volvy) | ||
| - [An Ethereum block](https://explore.ipld.io/#/explore/bagiacgzah24drzou2jlkixpblbgbg6nxfrasoklzttzoht5hixhxz3rlncyq) | ||
| - [XKCD archives](https://explore.ipld.io/#/explore/QmdmQXB2mzChmMeKY47C43LxUdg1NDJ5MWcKMKxDu7RgQm) |
There was a problem hiding this comment.
Example list changed, and some of these no longer work. Main landing page lists up-to-date examples, so removing them from here, saving screen estate.
2color
approved these changes
Oct 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Added glossary links + removed potential footguns.