Handle case insensitive paths
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 28
- Forks
- 25
- Avg merge
- 4d 22h
- Merged PRs (30d)
- 20
Description
Due to URLs being case sensitive a mistake was made in several endpoint that should have been camel case to separate words. e.g. timeseries should be timeSeries or TimeSeries.
Unfortunately we cannot just change them as it would break most existing usage. Javalin 5+ does support case-insensitive path matching.
Thus this should be fixed at such time as we can migrate to javalin 5.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the endpoint routing and the project's current Javalin version. When migration to Javalin 5 is possible, verify that existing paths such as timeseries continue to work while case-insensitive matching supports the intended camel-case forms; confirm the affected endpoints through the API's existing checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- api
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100