search

getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.87k stars 1.55k forks source link

Requiring within PhantomJS Script #395

Closed abhillman closed 6 years ago

abhillman commented 8 years ago

Thank you kindly for your time! It would be real nice to be able to use this with PhantomJS. Is this currently functional?

aryeh@abhillman-macbook-pro:/tmp/test$ phantomjs --version
1.9.8

aryeh@abhillman-macbook-pro:/tmp/test$ curl -o raven.js https://cdn.ravenjs.com/1.1.22/raven.min.js
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 15355  100 15355    0     0  20323      0 --:--:-- --:--:-- --:--:-- 20310

aryeh@abhillman-macbook-pro:/tmp/test$ node raven.js 

aryeh@abhillman-macbook-pro:/tmp/test$ phantomjs
phantomjs> require './tmp/test/raven.js'
Parse error

Perhaps I need to wrap the library with a loader to make this copacetic for phantomjs?

mattrobenolt commented 8 years ago

I have literally no idea what would be required in supporting this.

For my context, what's the use case here? And do you have any idea what would be needed?

/cc @benvinegar

kamilogorek commented 6 years ago

We don't support PhantomJS

benvinegar commented 6 years ago

We don't support PhantomJS

Well, that's not necessarily true. Some users use Raven.js + PhantomJS in part of their automated testing. But I think for this issue, which is 2 years old, has no actionable information, and no one else has surfaced, we can safely close.