Skip to content
This repository was archived by the owner on Nov 25, 2024. It is now read-only.

Conversation

@kegsay
Copy link
Member

@kegsay kegsay commented Feb 3, 2017

This automatically does JSON and logs with request IDs because it uses matrix-org/util. It looks like:

$ BIND_ADDRESS=:7878 ./bin/clientapi 
INFO[0000] Starting clientapi                           
INFO[0005] Incoming request                              req.id=x1slkZ1Mrk3B req.method=GET req.path="/sync"
INFO[0005] Doing stuff...                                req.id=x1slkZ1Mrk3B req.method=GET req.path="/sync"
INFO[0005] Responding with error                         error="Not implemented yet: 404: " req.id=x1slkZ1Mrk3B req.method=GET req.path="/sync"
INFO[0009] Incoming request                              req.id=ioZU9NanGplb req.method=GET req.path="/send"
INFO[0009] Doing stuff...                                req.id=ioZU9NanGplb req.method=GET req.path="/send"
INFO[0009] Responding with error                         error="Not implemented yet: 404: " req.id=ioZU9NanGplb req.method=GET req.path="/send"

Obviously the endpoints are stubs for now, but this gives the skeleton for how new APIs will be added.

Copy link
Contributor

@NegativeMjark NegativeMjark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kegsay kegsay merged commit 5b5c209 into master Feb 3, 2017
@NegativeMjark NegativeMjark deleted the kegan/client-webserver branch February 5, 2017 14:05
genofire pushed a commit to genofire/dendrite that referenced this pull request Dec 14, 2021
kuhnchris added a commit to kuhnchris/dendrite that referenced this pull request Jul 3, 2023
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 this pull request may close these issues.

3 participants