balderdashy/sails

How to handle a route with a dot in route param

Open

#6.954 geöffnet am 25. Feb. 2020

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (22.778 Stars) (1.953 Forks)batch import
does this answer your question?help wanted

Beschreibung

Node version: 10.19.0 Sails version (sails): 1.2.3

Hi,

I have a controller which serves an image. I am trying to get this route to work

/content/26/image.png

It responses 404 but work with

/content/26/image.png/

I got it to work with a custom inline route ( fn: async function (req, res)) but I want access control for this route.

Many thanks in advance and kindest regards

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.jsSails framework
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Investigate how Sails' router handles dots in route parameters by examining the source code in `lib/router/` and related configuration. Check if there is an existing option to allow dots in param values, and consider submitting a fix that updates the route matching regex or adds a flag. Review comments in the issue for any maintainer suggestions.