-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
minor suggestions #4
Comments
Thank you for letting me know! I definitely will work on:
These are extremely useful tips for a beginner like me, thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for this great API! With a few minor exceptions, it ran out-of-the-box!
I ran this API locally, but because the OpenAPI spec hard-codes the
poland-public-transport-api.azurewebsites.net
publicly deployed instance, and because that instance is currently down, the local OpenAPI playground doesn't work.https://github.com/konhi/poland-public-transport-api/blob/main/src/public/v1/openapi.json#L19
Currently, there are two logging formats mixed: JSON-based general logs about the server starting etc., and CLF for the actual requests.
What do you think about unifying them into one format, so that people running instances can process all logs with one set of tooling?
The text was updated successfully, but these errors were encountered: