openstate / openstate/stembureaus
Gemeentenaam in URL met andere casing redirecten naar canonical URL met hoofdletters
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 8
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
We schrijven gemeentenamen met hoofdletters in onze URLs, bv. https://waarismijnstemlokaal.nl/s/Eijsden-Margraten. Sinds deze commit staan we echter ook varianten toe met andere casing, bv:
- https://waarismijnstemlokaal.nl/s/eijsden-margraten
- https://waarismijnstemlokaal.nl/s/EiJsDen-MarGraTeN
Dit geeft echter problemen met onze nieuwe cache busting, want die purged enkel de canonical URL https://waarismijnstemlokaal.nl/s/Eijsden-Margraten en niet alle verschillende varianten qua casing. Daardoor blijft oude data te zien als iemand handmatig een URL met andere casing intypt.
We kunnen al deze URL's een key meegeven in Nginx, zodat we later op basis van die key alle gecachte varianten kunnen purgen. Simpeler is het om varianten van casing te redirecten naar onze canonical URL.
Contributor guide
No contributing guide indexed for this repository
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 tracing the route handling /s/ and read the cache-busting change in pull request #233. Check how mixed-case municipality names are currently accepted, then verify that non-canonical casing redirects to the canonical URL and that the canonical cache-busting behavior remains correct.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- nginx, python
- Domain
- backend, web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100