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

"Invalid sequence tag" for signing key GLS1111111111111111111111111111111114T1Anm #12

Closed
vikxx opened this Issue Oct 19, 2017 · 8 comments

Comments

Projects
None yet
7 participants
@vikxx

vikxx commented Oct 19, 2017

Broadcasting any other keys works well, but
broadcasting a GLS1111111111111111111111111111111114T1Anm key by calling golos.broadcast.witnessUpdate

causes an error: Invalid sequence tag

image

@On1x

This comment has been minimized.

On1x commented Nov 14, 2017

Up, need to fix it!

@Jolly-Pirate

This comment has been minimized.

Jolly-Pirate commented Nov 23, 2017

Any progress on this issue yet?

@asuleymanov

This comment has been minimized.

asuleymanov commented Mar 6, 2018

Если я правильно понял описание из кода

block_signing_key The new block signing public key.  The empty string disables block production.

То для того чтобы остановить просто нужно указать там пустоту.

@asuleymanov

This comment has been minimized.

asuleymanov commented Jul 4, 2018

Как я понимаю воз и ныне там.
Я подозреваю это из-за того что документация слабовата.
Пойдем пинать Core.

@asuleymanov

This comment has been minimized.

asuleymanov commented Jul 4, 2018

Итак товарищи я нашел решение данной проблемы. По крайней мере на GO.

При получении данного ключа я делаю вот такую штуку.

if op.BlockSigningKey == "GLS1111111111111111111111111111111114T1Anm" {
		var dst []byte
		for i := 0; i < 33; i++ {
			dst = append(dst, 0)
		}
		enc.Encode(dst)
	} else {
		enc.EncodePubKey(op.BlockSigningKey)
	}

Попробую разъяснить.
Я создаю массив байт размером 33 и заполняю его нулями и именно это значение отправляю на сериализацию. А в JSON уходит непосредственно сам ключ.

Ну вот решение я нашел. Теперь дело за малым чтобы его у Вас в JS реализовали.

Если что непонятно готов разъяснить.

@vikxx

This comment has been minimized.

vikxx commented Jul 4, 2018

@b1acksun b1acksun self-assigned this Jul 5, 2018

@b1acksun b1acksun added the bug label Jul 5, 2018

@b1acksun

This comment has been minimized.

b1acksun commented Jul 5, 2018

fixed in 0.7.2

@netuoso

This comment has been minimized.

netuoso commented Jul 13, 2018

Man golos is on the ball. Might need to learn Russian and start contributing there more!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment