-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
List available schemas in 404 response to /schemas
#3469
List available schemas in 404 response to /schemas
#3469
Conversation
428dae0
to
92c8871
Compare
I think it is a good idea to point I don't think we should publish a human-readable list at |
Thanks for your response @tigrannajaryan - see #3468 (comment) for a clarification of the current situation ( |
5051312
to
02e8a37
Compare
3a7c207
to
07acb2d
Compare
Converting to draft until we reach a consensus at #3468. In the meantime, I'm testing the option of returning a 404 status for the schema index page, while still providing the list of schemas in human readable form. |
Ok, I'm going to split this up so as to not hold back the individual fixes. Here's the first: |
Here's the other feature factored out: I'll repurpose this PR to be only about the index page. |
latest
redirect, index page, fix content-type07acb2d
to
5e9f962
Compare
5e9f962
to
02d1844
Compare
/schemas
02d1844
to
ac33358
Compare
Closing in favor of: |
/schemas
is the top 404 #3468 by adding an index page, if semconv and spec folks agree to this feature.Note that the list is autogenerated from
content-modules/semantic-conventions/schemas
Fixes Is there an index for all available schema? #3256moved to Redirect/schemas/latest
to latest schema #3475Fixes Ensure that schema files are served with content-type application/yaml #3416moved to Schema content-type fix #3474Preview: https://deploy-preview-3469--opentelemetry.netlify.app/schemas/
Screenshot (mobile)