Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent recursive failsafes #909

Merged
merged 6 commits into from
Oct 31, 2019
Merged

Prevent recursive failsafes #909

merged 6 commits into from
Oct 31, 2019

Conversation

johnsyweb
Copy link
Contributor

@johnsyweb johnsyweb commented Oct 31, 2019

Context

See #632

Change

Confirmation

E, [2019-10-31T04:39:43.508233 #1135] ERROR -- : [Rollbar] Sending failsafe response due to SocketError: "Failed to open TCP connection to api.rollbar.com:443 (getaddrinfo: Temporary failure in name resolution)" in /usr/lib/ruby/2.5.0/net/http.rb:939:in `rescue in block in connect': error in process_item
I, [2019-10-31T04:39:43.508357 #1135]  INFO -- : [Rollbar] Scheduling item
E, [2019-10-31T04:39:43.508698 #1135] ERROR -- : [Rollbar] Item: #<Rollbar::Item:0x0000000004c314b0>
I, [2019-10-31T04:39:43.508853 #1135]  INFO -- : [Rollbar] Sending item
E, [2019-10-31T04:39:43.509462 #1135] ERROR -- : [Rollbar] Error processing the item: SocketError, Failed to open TCP connection to api.rollbar.com:443 (getaddrinfo: Temporary failure in name resolution). Item: {"access_token"=>"*******", "data"=>{:level=>"error", :environment=>"staging", :body=>{:message=>{:body=>"Failsafe from rollbar-gem. SocketError: \"Failed to open TCP connection to api.rollbar.com:443 (getaddrinfo: Temporary failure in name resolution)\" in /usr/lib/ruby/2.5.0/net/http.rb:939:in `rescue in block in connect': error in process_item"}}, :notifier=>{:name=>"rollbar-gem", :version=>"2.22.1"}, :custom=>{:orig_uuid=>nil, :orig_host=>nil}, :internal=>true, :failsafe=>true}}
E, [2019-10-31T04:39:43.509560 #1135] ERROR -- : [Rollbar] Item has already failed. Not re-raising

@waltjones
Copy link
Contributor

@johnsyweb Looks good, thank you!

@waltjones waltjones merged commit af4bbb2 into rollbar:master Oct 31, 2019
@johnsyweb
Copy link
Contributor Author

Thanks for the quick turnaround, @waltjones, much appreciated . When do you expect to release the next version of the gem, please?

@waltjones
Copy link
Contributor

One to two weeks. There are a handful of other things that need to go in. One less now though. ;)

@johnsyweb
Copy link
Contributor Author

Perfect. Thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants