search

sensu / sensu-admin

An admin webui for Sensu
MIT License
86 stars 50 forks source link

Allow dot in hostnames #65

Closed hmarr closed 11 years ago

hmarr commented 11 years ago

This fixes #63. The issue was caused by hostnames that contain periods. The default Rails regex for matching parameters excludes dots, this modifies the constraint the for the client parameter to allow dots.

I also restructured the routes to make use of scopes, to avoid repeating the first part of the path, as well as the additional constraint, in each entry.

joshpasqualetto commented 11 years ago

This is perfect, thank you for not only fixing the problem but actually organizing the routes sanely.