Sinatra runs twice
Reported by Mike Perham | July 11th, 2008 @ 09:23 AM | in 0.3.0 Sammy
When I start Sinatra and kill it with Ctrl-C, it starts right back up again. I have to kill it a second time for it to actually quit. This is true only in development and only after I've made an initial request. For this reason, I suspect reloading is at fault somehome.
This issue is in edge and was not present in 0.2.2.
Comments and changes to this ticket
-
Blake Mizerany July 11th, 2008 @ 10:49 AM
awesome. It's an undocumented feature. It's making sure everything is being run. :)
I've seen this before as well. Let's see if it happens anymore after this weeks release.
-
Ryan Tomayko September 7th, 2008 @ 06:28 PM
- → State changed from “new” to “resolved”
I haven't seen this in recent versions so I'm thinking something was likely fixed? Leave a comment if you're still seeing the same behavior and we'll re-open and take a look for the 0.3 series.
-
-
Ryan Tomayko September 11th, 2008 @ 03:01 PM
- → State changed from “resolved” to “open”
Oh well. reopened.
Can you give me any more info about the app? Are you running it w/ ruby or something like Thin? Setting any options or anything?
-
Mike Perham September 11th, 2008 @ 03:07 PM
See ticket-37.zip which is a small project which reproduces the problem. Note that we're trying to use the Daemons library to control Sinatra start/stop.
Unzip the file, cd to repro and run "bin/collector run". Hit Ctrl-C.
-
-
Ryan Tomayko September 11th, 2008 @ 03:08 PM
- → Assigned user changed from “Blake Mizerany” to “Ryan Tomayko”
-
Mike Perham September 11th, 2008 @ 03:35 PM
- → Assigned user changed from “Ryan Tomayko” to “Blake Mizerany”
Ryan, it turns out this was my fault. A refactoring of the code in collector.rb meant that the proper options weren't being loading into Sinatra's default_options. Notably this line is fixed:
Sinatra::Application.default_options.merge!(COLLECTOR_OPTIONS) if Object.const_defined? 'COLLECTOR_OPTIONS' -
Ryan Tomayko September 11th, 2008 @ 06:17 PM
- → State changed from “open” to “resolved”
- → Assigned user changed from “Blake Mizerany” to “Ryan Tomayko”
Nice.
Please Login or create a free account to add a new comment.
You can update this ticket by sending an email to from your email client. (help)
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
Blake Mizerany