jhawthorn / vernier

📏 next generation CRuby profiler
https://vernier.prof/
MIT License
718 stars 15 forks source link

Middleware: access Rack GET params as strings not symbols #77

Closed bensheldon closed 2 weeks ago

bensheldon commented 2 weeks ago

While integrating this into an app, I noticed these are accessed as symbols, which I'm pretty sure won't work.

A few lines up from the diff in this PR, the request.GET.has_key?("vernier") is accessed as a string, so I'm pretty sure this is the right way to do it.