Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Querying a non-existent user's full profile should have an error code of M_NOT_FOUND, not M_UNKNOWN #1435

Closed
turt2live opened this issue Jul 19, 2018 · 2 comments
Closed

Querying a non-existent user's full profile should have an error code of M_NOT_FOUND, not M_UNKNOWN #1435

turt2live opened this issue Jul 19, 2018 · 2 comments
Labels
improvement A suggestion for a relatively simple improvement to the protocol s2s Server-to-Server API (federation)

Comments

@turt2live
Copy link
Member

The current synapse behaviour is to return M_UNKNOWN, therefore that will be landing in the spec. This should be changed to be M_NOT_FOUND.

Related: #1434
@turt2live turt2live added improvement A suggestion for a relatively simple improvement to the protocol s2s Server-to-Server API (federation) labels Jul 19, 2018
turt2live added a commit to turt2live/matrix-doc that referenced this issue Jul 20, 2018
@turt2live
Combine all queries into their own section
f88173e 
This removes the Directory and Profile sections, instead opting to document them as Queries. 

The behaviour of profile queries is based on Synapse's behaviour. A few issues have been opened to improve the behaviour:
* matrix-org#1434
* matrix-org#1435
* https://github.com/matrix-org/matrix-doc/issues/1436
* https://github.com/matrix-org/matrix-doc/issues/1437

This fixes matrix-org#1404
@turt2live turt2live mentioned this issue Jul 20, 2018
Merged
@richvdh
Copy link
Member

richvdh commented Jul 23, 2018

The current synapse behaviour is to return M_UNKNOWN, therefore that will be landing in the spec

must it? I find no value in speccing such a thing.

@turt2live
Copy link
Member Author

moving to synapse as a bug

This was referenced Jul 23, 2018
Closed
Closed
turt2live added a commit to turt2live/matrix-doc that referenced this issue Jul 24, 2018
@turt2live
Combine all queries into their own section
1873f2a 
This removes the Directory and Profile sections, instead opting to document them as Queries. 

The behaviour of profile queries is based on Synapse's behaviour. A few issues have been opened to improve the behaviour:
* matrix-org#1434
* matrix-org#1435
* https://github.com/matrix-org/matrix-doc/issues/1436
* https://github.com/matrix-org/matrix-doc/issues/1437

This fixes matrix-org#1404
turt2live added a commit to turt2live/matrix-doc that referenced this issue Jul 26, 2018
@turt2live
Combine all queries into their own section
0ddf578 
This removes the Directory and Profile sections, instead opting to document them as Queries. 

The behaviour of profile queries is based on Synapse's behaviour. A few issues have been opened to improve the behaviour:
* matrix-org#1434
* matrix-org#1435
* https://github.com/matrix-org/matrix-doc/issues/1436
* https://github.com/matrix-org/matrix-doc/issues/1437

This fixes matrix-org#1404
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement A suggestion for a relatively simple improvement to the protocol s2s Server-to-Server API (federation)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@turt2live @richvdh