Check that master key is returned when querying device for first time#1166
Check that master key is returned when querying device for first time#1166erikjohnston merged 5 commits intodevelopfrom
Conversation
|
There seem to be a mix of tabs vs. spaces going on here. |
WHY WOULD YOU DO THIS TO ME VIM WHY 😭 |
|
|
||
| loggers => { | ||
| synapse => { | ||
| level => "INFO" |
There was a problem hiding this comment.
Was this just for temporary debugging?
There was a problem hiding this comment.
Oh blast, I carefully didn't add this, and then clearly did a git commit -a.
|
https://github.com/matrix-org/sytest/runs/4091934296?check_suite_focus=true Looks like #1164. Edit: yes, it does. Bugger. https://github.com/matrix-org/sytest/runs/4091934438?check_suite_focus=true failed to install dendrite. |
|
/me waits to see how else he has messed up this PR |
DMRobertson
left a comment
There was a problem hiding this comment.
Seems reasonable; one question
| assert_json_keys( $content->{master_keys}, $user2_id ); | ||
| assert_json_keys( $content->{master_keys}->{$user2_id}, "keys"); | ||
| assert_json_keys( $content->{master_keys}->{$user2_id}{keys}, | ||
| "ed25519:nqOvzeuGWT/sRx3h7+MHoInYj3Uk2LD/unI9kDYcHwk"); |
There was a problem hiding this comment.
Should we also expect to see this in the second call to matrix_get_e2e_keys on +685?
There was a problem hiding this comment.
Yes we should. I suppose its worth checking that its still there
See matrix-org/synapse#11234