Skip to content

Add Forbole validator info - #64

Open
kwunyeung wants to merge 2 commits into
atomone-hub:mainfrom
kwunyeung:add-forbole-detail
Open

Add Forbole validator info#64
kwunyeung wants to merge 2 commits into
atomone-hub:mainfrom
kwunyeung:add-forbole-detail

Conversation

@kwunyeung

Copy link
Copy Markdown
Collaborator

No description provided.

@kwunyeung kwunyeung changed the title Added Forbole validator info Add Forbole validator info Nov 7, 2024
@kwunyeung
kwunyeung requested a review from Ticojohnny November 7, 2024 09:32
@VT-Cosmos

Copy link
Copy Markdown
Contributor

Hi @kwunyeung - Thank you for submitting your details.
Can you specify the country please.

@kwunyeung

Copy link
Copy Markdown
Collaborator Author

@VT-Cosmos the region info is updated.

@VT-Cosmos

Copy link
Copy Markdown
Contributor

Thank you @kwunyeung

@tbruyelle tbruyelle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the submission — it can't be verified as it stands.

The transaction was not signed by your validator's account.

  • signer: atone14kn0kk33szpwus9nh8n87fjel8djx0y04mvfpe
  • Forbole account: atone1ysvhj4h3z6szgwauuhvytxks9k6s6ktufweqt5

The signing account does delegate to Forbole, so it looks like a team wallet rather than the validator key. That is not enough — anyone can delegate.

The transfer has to come from the validator account itself (atone1ysvhj4h3z6szgwauuhvytxks9k6s6ktufweqt5), since that is what proves the submission comes from the operator and not from someone who merely knows the moniker.

validator-info.md changed after you opened this: #170 added every verified entry, so this branch now conflicts with main. Update your branch and add your row at the end of the table.

The three checks every submission has to pass, and the full audit of all 128 submissions, are in #168.

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.

3 participants