Skip to content

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Aug 5, 2016

Future change might involve renaming current_environment to environment and environments to something that looks like like the core attribute.

@getsentry/ruby


This change is Reviewable

Future change might involve renaming current_environment to environment and environments to something that looks like like the core attribute.

@getsentry/ruby
@nateberkopec nateberkopec merged commit 16cd516 into master Aug 5, 2016
@dcramer dcramer deleted the send-environment branch August 5, 2016 19:58
data[:culprit] = @culprit if @culprit
data[:server_name] = @server_name if @server_name
data[:release] = @release if @release
data[:environment] = @envirionment if @environment
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like @envirionment is misspelled.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was fixed in a followup commit

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, thanks! (and sorry for not looking more closely)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't sweat it, I merged it and missed the same thing! 😆

alexford pushed a commit to alexford/raven-ruby that referenced this pull request Jan 1, 2017
Usage:

Without the header option, both request / response headers are logged (backwards compatibility).

```ruby
Faraday.new do |c|
  c.response :logger, Logger.new(STDOUT)
end

Faraday.new do |c|
  c.response :logger, Logger.new(STDOUT), headers: false
end

Faraday.new do |c|
  c.response :logger, Logger.new(STDOUT), headers: { request: false }
end

Faraday.new do |c|
  c.response :logger, Logger.new(STDOUT), headers: { response: false }
end
```
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.

4 participants