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

How about supporting uppercased route HTTP in Ring handler method keywords? #658

Open
gerdint opened this issue Dec 29, 2023 · 2 comments
Open

Comments

@gerdint
Copy link

gerdint commented Dec 29, 2023

I.e:

["/api-url" {:POST api/do-this
             :parameters { ... }
             :summary "Do this"}]

This would make the method keyword stand out among the other route keywords (and also map to the HTTP spec better I think).

@ikitommi
Copy link
Member

interesting idea. But this would be a breaking change unless the lowercase keys would also be supported.

@gerdint
Copy link
Author

gerdint commented Dec 29, 2023

Yes the intention was to have both be supported.

@gerdint gerdint changed the title How about supporting uppercased route HTTP method keywords? How about supporting uppercased route HTTP in Ring handler method keywords? Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants