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.

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

Add /.well-known/schema-discovery route for queries from TWINTIP #82

Closed
chutium opened this issue Oct 29, 2015 · 8 comments
Closed

Add /.well-known/schema-discovery route for queries from TWINTIP #82

chutium opened this issue Oct 29, 2015 · 8 comments

Comments

@chutium
Copy link

chutium commented Oct 29, 2015

many exceptions regularly appeared on our applications, we find they were caused by the queries from TWINTIP (https://docs.stups.io/en/latest/components/twintip.html)

we need to create a dummy response of "/.well-known/schema-discovery" for each applications, most of them are using connexion.

so it will be nice if connexion can handle those TWINTIP queries internally, or provide a template or something to users, so that we can easily fill some text and connexion generate the response internally.

@elezar
Copy link

elezar commented Oct 29, 2015

Could one add the routes when an API is added using a call to add_api returning the required information from the api?

What happens when one calls add_api multiple times (specifying different .yaml files each time)?

@hjacobs
Copy link
Contributor

hjacobs commented Oct 29, 2015

@chutium I think adding (at least allowing "opt-in") the schema-discovery URL is a good idea

@hjacobs
Copy link
Contributor

hjacobs commented Oct 29, 2015

For background explanation: the "/.well-known/schema-discovery" URL (https://docs.stups.io/en/latest/components/twintip.html) is our Zalando STUPS way of discovering Swagger API definitions. The STUPS TWINTIP service is regularly scanning all deployed application endpoints to find Swagger API definitions.

@hjacobs
Copy link
Contributor

hjacobs commented Oct 29, 2015

FYI: There is also an ongoing discussion about allowing multiple Swagger API definitions per service, see https://github.com/zalando-stups/twintip-crawler/issues/5

@hjacobs
Copy link
Contributor

hjacobs commented Oct 29, 2015

@chutium I now see your problem more clearly: I guess the real problem is Connexion logging HTTP 401 (unauthenticated) as "ERROR", we probably should reconsider and use "INFO" log level (as it's a client error and should not spam the server logs)

@hjacobs
Copy link
Contributor

hjacobs commented Oct 29, 2015

@chutium see #83

@rafaelcaricio
Copy link
Collaborator

Can we close this issue?

@hjacobs
Copy link
Contributor

hjacobs commented Feb 15, 2016

Yes, let's close it.

@hjacobs hjacobs closed this as completed Feb 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants