Skip to content

Conversation

ckampfe
Copy link
Contributor

@ckampfe ckampfe commented Sep 17, 2018

Hey there, I'm getting ** (Mix) Could not start application poison: could not find application file: poison.app. My app uses Jason, so I'm using Jason with sentry as well. Maybe poison isn't needed in the app start list? Thanks!

@mitchellhenke
Copy link
Contributor

Thank you!

@mitchellhenke mitchellhenke merged commit 8405c38 into getsentry:master Sep 17, 2018
@ckampfe ckampfe deleted the remove-poison-from-apps-list branch September 17, 2018 21:15
@nathany
Copy link

nathany commented Oct 1, 2018

Thanks @ckampfe.

@mitchellhenke Will there be a 7.0.1 release soon that includes this fix?

@nathany
Copy link

nathany commented Oct 1, 2018

Btw, I am seeing this warning when using master.

==> sentry
Compiling 13 files (.ex)
Generated sentry app
warning: failed to load Poison for Phoenix JSON encoding
(module Poison is not available).

Ensure Poison is loaded from your deps in mix.exs, or
configure an existing encoder in your Mix config using:

    config :phoenix, :json_library, MyJSONLibrary


  (phoenix) lib/phoenix.ex:40: Phoenix.start/2
  (kernel) application_master.erl:277: :application_master.start_it_old/4

@mitchellhenke
Copy link
Contributor

@nathany yep, I just released 7.0.1

Your warning is coming from Phoenix it looks like?

@nathany
Copy link

nathany commented Oct 1, 2018

Thanks.

It appears to be from Phoenix -- specifically when compiling sentry, however. Not sure if that's an issue to worry about, just thought I'd mention it.

@mitchellhenke
Copy link
Contributor

@nathany which version of Phoenix?

@nathany
Copy link

nathany commented Oct 1, 2018

I'm using master (1.4). I just updated from 29c24c9828aa346155e3157f4dec70842da2a58a to 2ddd0ee46ddf49e44db0ec876dc9c4cff940f2d2 to confirm that it is still displaying the warning while compiling.

I tested raise "oops" in a controller and everything is working fine.

@mitchellhenke
Copy link
Contributor

ah ok, cool

After testing a bit on a new Phoenix project, it looks like you may need to add config :phoenix, :json_library, Jason to your config/config.exs. Without that line, I get the warning 🙂

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.

3 participants