Skip to content
This repository was archived by the owner on Jan 29, 2022. It is now read-only.

Fix a http body param bug#88

Merged
alexjfisher merged 1 commit intomasterfrom
fix_environment_cont
Feb 18, 2019
Merged

Fix a http body param bug#88
alexjfisher merged 1 commit intomasterfrom
fix_environment_cont

Conversation

@dhollinger
Copy link
Copy Markdown
Member

Fixed a bug where one of the parameters sent to a method was named
body. This was an issue as the body method is built into Rack and
was causing the application to become confused are fail on API calls.

Updated and organized .gitignore.

Gemfile.lock updated as part of a bundle install

Fixed a bug where one of the parameters sent to a method was named
`body`. This was an issue as the `body` method is built into Rack and
was causing the application to become confused are fail on API calls.

Updated and organized .gitignore.

Gemfile.lock  updated as part of a bundle install
@dhollinger dhollinger self-assigned this Feb 18, 2019
@dhollinger dhollinger added the bug label Feb 18, 2019
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 52.189% when pulling bafc570 on fix_environment_cont into f80c0ec on master.

@runejuhl
Copy link
Copy Markdown

LGTM

@alexjfisher alexjfisher merged commit 8583d32 into master Feb 18, 2019
alexjfisher added a commit to alexjfisher/puppet_webhook that referenced this pull request Feb 18, 2019
@alexjfisher alexjfisher deleted the fix_environment_cont branch February 18, 2019 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants