matrix-org/synapse

Follow the spec regarding /client/r0/keys/changes

Aperta

#3058 aperta il 3 apr 2018

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)Python (2196 fork)batch import
A-Spec-ComplianceS-TolerableT-Defectgood first issue

Metriche repository

Star
 (11.713 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Description

Moving the discussion from the spec repo (https://github.com/matrix-org/matrix-doc/issues/1170).

A request to the endpoint will succeed even if it's missing the to query parameter which is marked as mandatory.

Steps to reproduce

curl -XGET "https://matrix.org/_matrix/client/r0/keys/changes?access_token=${TOKEN}&from=${NEXT_BATCH}"

Replacing TOKEN and NEXT_BATCH with a valid access token and a next_batch token returned by sync, respectively.

Expected behavior.

The endpoint should throw an error.

Version information

Tested with synapse v0.26 & with matrix.org

Guida contributor

Direzione di ricerca
Esamina l'implementazione dell'endpoint /keys/changes in Synapse (probabilmente in synapse/rest/client/keys.py) per vedere come viene convalidato 'from' ma non 'to'. Assicurati che 'to' sia trattato come obbligatorio secondo le specifiche Matrix (problema matrix doc #1170). Aggiungi la convalida necessaria, simile ad altri parametri di query obbligatori nella codebase.
Tech stack
python
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70