Skip to content

Updating the example of invoking actions with Go#21

Open
xwjdsh wants to merge 1 commit intomasterfrom
feat/update-invoke-action-with-go
Open

Updating the example of invoking actions with Go#21
xwjdsh wants to merge 1 commit intomasterfrom
feat/update-invoke-action-with-go

Conversation

@xwjdsh
Copy link
Contributor

@xwjdsh xwjdsh commented Jul 31, 2023

  • Add checksum data to memo

@xwjdsh xwjdsh requested a review from lyricat July 31, 2023 13:36
@xwjdsh xwjdsh marked this pull request as ready for review July 31, 2023 13:36
panic(err)
}
return base64.StdEncoding.EncodeToString(enc.Bytes())
return base64.StdEncoding.EncodeToString(checksum.Sha256(enc.Bytes()))
Copy link
Contributor

Choose a reason for hiding this comment

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

这个 checksum.Sha256 是把整个 payload 丢进去,出来的是 payload + sum 吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

结果是 sum,应该 append 的,修改了。

@xwjdsh xwjdsh force-pushed the feat/update-invoke-action-with-go branch from 03e52b3 to 1774056 Compare July 31, 2023 16:07
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.

2 participants