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

corrupted ipns entry in local cache #3124

Open
whyrusleeping opened this issue Aug 25, 2016 · 2 comments
Open

corrupted ipns entry in local cache #3124

whyrusleeping opened this issue Aug 25, 2016 · 2 comments
Labels
help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) status/deferred Conscious decision to pause or backlog topic/dht Topic dht topic/ipns Topic ipns

Comments

@whyrusleeping
Copy link
Member

I was able to break my local ipns publishing:

➜  ~ ipfs name publish <some hash>
Error: proto: required field "{Unknown}" not set

I did this by first publishing an entry normally, then:

ipfs dht get /ipns/<my peer ID> > ipns-data
cat ipns-data | ipfs dht put /ipns/<my peer ID>

Something must not be quite right here in the round tripping of the record.

This also appears to make it impossible for me to publish new ipns values for the time being.

@whyrusleeping whyrusleeping added kind/bug A bug in existing code (including security flaws) topic/dht Topic dht topic/ipns Topic ipns labels Aug 25, 2016
@jbenet
Copy link
Member

jbenet commented Aug 26, 2016

That would be a great test case of you can get a repro
On Thu, Aug 25, 2016 at 14:59 Jeromy Johnson notifications@github.com
wrote:

I was able to break my local ipns publishing:

➜ ~ ipfs name publish
Error: proto: required field "{Unknown}" not set

I did this by first publishing an entry normally, then:

ipfs dht get /ipns/ > ipns-data
cat ipns-data | ipfs dht put /ipns/

Something must not be quite right here in the round tripping of the record.

This also appears to make it impossible for me to publish new ipns values
for the time being.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#3124, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIcoQT8Rok7HAR212R3_K_QFOcVbsfOks5qjeYOgaJpZM4JtZHq
.

@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label Sep 14, 2016
@whyrusleeping whyrusleeping added the status/deferred Conscious decision to pause or backlog label Dec 1, 2016
@Stebalien
Copy link
Member

Well, the good news is that this now returns an error on put. The bad news is that get is still broken.

Stebalien added a commit that referenced this issue Apr 30, 2019
Otherwise, it seems that something is treating this as UTF8 and normalizing it?

fix half of #3124

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) status/deferred Conscious decision to pause or backlog topic/dht Topic dht topic/ipns Topic ipns
Projects
None yet
Development

No branches or pull requests

3 participants