Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

Backport "correct request body for the /keys/query endpoint" to support #285

Closed
jdauphant opened this issue Jul 3, 2022 · 3 comments · Fixed by #287
Closed

Backport "correct request body for the /keys/query endpoint" to support #285

jdauphant opened this issue Jul 3, 2022 · 3 comments · Fixed by #287
Assignees
@jdauphant
Labels
P1 Priority 1 synapse mainlining web tchap-web

Comments

@jdauphant
Copy link
Contributor

The last version of synapse enforce the "/keys/query endpoint" API.
We need to backport : matrix-org/matrix-js-sdk#1307 in matrix-js-sdk
@jdauphant jdauphant added web tchap-web P1 Priority 1 labels Jul 3, 2022
@jdauphant jdauphant self-assigned this Jul 10, 2022
@jdauphant jdauphant mentioned this issue Jul 10, 2022
Merged
@estellecomment
Copy link
Contributor

From what I can tell, our js-sdk is based on element's v2.2.0. The PR we have to backport got addded to v5.3.0. It's too far forward for us to merge our js-sdk with element's v5.3.0.

@jdauphant
Copy link
Contributor Author

I have done more than the backport. I have checked the code where this api call is made. They was just a transformation from JS to TS.
The more trouble was to adapte the tests.

@jdauphant jdauphant mentioned this issue Jul 25, 2022
Closed
8 tasks
@mcalinghee mcalinghee mentioned this issue Jul 25, 2022
Closed
@mcalinghee
Copy link
Contributor

Duplicate with #275

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

@jdauphant jdauphant

Labels
P1 Priority 1 synapse mainlining web tchap-web
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use the correct request body for the /keys/query endpoint tchapgouv/tchap-web-v2
3 participants
@estellecomment @jdauphant @mcalinghee