Skip to content
This repository has been archived by the owner on Jun 21, 2020. It is now read-only.

In simulation mode, the quote is returned in ASCII instead of bytes #150

Closed
lacabra opened this issue May 8, 2019 · 0 comments · Fixed by #151
Closed

In simulation mode, the quote is returned in ASCII instead of bytes #150

lacabra opened this issue May 8, 2019 · 0 comments · Fixed by #151

Comments

@lacabra
Copy link
Contributor

lacabra commented May 8, 2019

Describe the bug

When running in simulation mode, p2p starts and requests core for GetRegistrationParams, core returns the report in ASCII, like the following output below:

core_1       | 16:14:37 [INFO] LOG DERIVE: Ok(GetRegistrationParams { result: RegistrationParams { signing_key: "407ae45a92d1198417a42ad6ec5f9eec77276a9e", report: "AgAAAAsAAADvAO8A7+/v77AzX9O8HMqPgE65imQgWS0Xb8iuFXC0gB8FSr/xIOM9SCDzN2rmsvIDTTt6S0ineAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABwAAAAAAAAADAAAAAAAAAN9u81wqrOF5YPWZjzF0mVNrJL0TKw9p9c/wi6cilZRoAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACD1xnnferKFHD2uvYqTXdDA8iZ22kCD5xw7h38CMfOngAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABAeuRaktEZhBekKtbsX57sdydqngAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAqAIAAO7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7ohSUx0tDfnWJzB9NGgBAADu7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u7u4IfgBHUB6MHQnVtq4HwdGl", signature: "" } })

Compare the above with the different output when running in Hardware mode, which is what p2p expects and processes (in which the report is in hex, as per what the docs specify).

core_1       | 16:21:56 [INFO] LOG DERIVE: Ok(GetRegistrationParams { result: RegistrationParams { signing_key: "4a99f192101dbff1731c8c945aa86c1d48722331", report: "7b226964223a223236373338363634323730393034393537333138323538373033343739353431343039393536222c2274696d657374616d70223a22323031392d30352d30385431363a32313a35352e393632303036222c2276657273696f6e223a332c22697376456e636c61766551756f7465537461747573223a2247524f55505f4f55545f4f465f44415445222c22706c6174666f726d496e666f426c6f62223a22313530323030363530343030304630303030303530353032303430313031303030303030303030303030303030303030303030383030303030393030303030303032303030303030303030303030304143373438333745314139424443334145323146394145333137374241353646344434414343313246334130304431454645463039464546374230384136384336353834314637463641393534314633344541304535443243394544434137383636423130304431413446393037333846463441383130334145314142354436433431222c22697376456e636c61766551756f7465426f6479223a2241674141414d634b414141484141594141414141414c417a58394f38484d715067453635696d5167575333354746756553724a357a34564f3951636d316663684167582f42502f2f41414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414277414141414141414141484141414141414141414f7a345256447273684c4c694c31365268344773646c722f317a706d3352744b3458506b57766845675066414141414141414141414141414141414141414141414141414141414141414141414141414141414141434431786e6e6665724b4648443275765971545864444138695a32326b434435787737683338434d664f6e674141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141424b6d6647534542322f38584d636a4a5261714777645348496a4d5141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141414141227d", signature: "99d46508f83f2a72643022020898653a14d2d73cbb9069501b7b4b0140e57eca84dc6123f2982f35e8f2a23c8ecafcfe4a0e6a46154094e3f8380f2ef7691225ba8072bbfccbd0fd7141e64a7d9ec0308ee6aea51e94f8a6d6773ac6fe450dec750d24c7a397e23f247a4a5dabcbda8549828415fdc9c8167329aa5d5da656d9fd741522ab61a34afb28a937d3ca68591326b65e046982fe856a14d41ec5c96d2d80141a26f1f0fceb54c7a90619135d94779d430e16ad3d52d75d4f797a22d6e0db33afa3f2a919c35f2fcf37fce0bfdd7e3022349af01b9c3a7352c20197975a936eb12d5a01684707602b51ea97ba600246f78d2975e60d6ace4be28aced9" } })

To Reproduce
Steps to reproduce the behavior:

  1. In the discovery-integration-tests repo, edit the following line: SGX_MODE=SW
  2. Launch the network with ./launch.bash and look at the network output

Expected behavior
The format of the report should be same in both Hardware and Simulation mode, as hexadecimal, following the specification.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant