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

Quickbooks::IntuitRequestException: : <?xml version="1.0"?> #392

Open
bcackerman opened this issue Aug 11, 2017 · 12 comments
Open

Quickbooks::IntuitRequestException: : <?xml version="1.0"?> #392

bcackerman opened this issue Aug 11, 2017 · 12 comments

Comments

@bcackerman
Copy link
Contributor

We just switched to Puma/Sidekiq and we get this error often as well when using quickbooks-ruby gem. Here's a short backtrace: https://cl.ly/ly83

Is this a bug with quickbooks-ruby?

@ruckus
Copy link
Owner

ruckus commented Aug 11, 2017

Possibly related to #393 - it looks like net/http is not thread-safe which might be the crux of this

https://stackoverflow.com/questions/3063088/is-rubys-nethttp-threadsafe

@bcackerman
Copy link
Contributor Author

@ruckus Is there a way we can not use net/http or make it threadsafe?

@ruckus
Copy link
Owner

ruckus commented Aug 11, 2017

I think we're going to have to switch to a threadsafe HTTP client like

@bcackerman
Copy link
Contributor Author

Would installing the gem here https://github.com/drbrain/net-http-persistent/ help in the meantime you feel @ruckus ?

@ruckus
Copy link
Owner

ruckus commented Aug 11, 2017

Although I am quite surprised that no one has encountered this issue before. Put another way: I find it hard to believe you're the first Puma/Sidekiq user to use this library.

I see you're on Ruby 2.4. Perhaps thats an issue? Is it reproducible on Ruby 2.3 under Puma/Sidekiq?

@drewish
Copy link
Contributor

drewish commented Aug 31, 2017

We've been getting this for a long time on ruby 2.3.1 in sidekiq jobs. I'd never bothered trying to track down the cause so I'm excited to see this thread!

@drewish
Copy link
Contributor

drewish commented Apr 10, 2018

@ruckus in the OAuth2 branch you're moving to Faraday. Would you consider pulling some of that work forward into a separate PR to help address these threading issues?

@briandiaz
Copy link

Any solution?

@ruckus
Copy link
Owner

ruckus commented Jul 19, 2018

@drewish Yes the 389-oauth2 branch does use Faraday but thats really indirectly by virtue of the dependent oauth2 gem using it.

This gem does not directly use net/http, its used internally by the oauth(1) gem.

Which is to say that its not so easy for this gem to swap out the internal HTTP library as really its internal to the underlying oauth gem.

@bcackerman
Copy link
Contributor Author

Are there plans to merge 389-oauth2 into master?

@greycampbell
Copy link

Hi all, I'm currently on quickbooks-ruby version 1.0.17 and am seeing this error periodically in my Sidekiq jobs. I've upgraded Faraday (1.10.0) and OAuth2 (1.4.9), and set the http adapter to net_http_persistent.

Is anyone else still seeing this error?

@ruckus
Copy link
Owner

ruckus commented Apr 16, 2022

@greycampbell oof, thats no bueno. im still in a single-threaded / process based environment, so I wouldn't see this. hopefully others that can chime in and provide advice/ideas/support.

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

No branches or pull requests

5 participants