Navigation Menu

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

[dev.icinga.com #13411] Missing API headers for X-HTTP-Method-Override #4829

Closed
icinga-migration opened this issue Dec 5, 2016 · 6 comments
Labels
area/documentation End-user or developer help bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/13411

Created by tofu00 on 2016-12-05 20:26:43 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2016-12-06 09:50:02 +00:00)
Target Version: 2.6.0
Last Update: 2016-12-06 09:50:02 +00:00 (in Redmine)

Icinga Version: v2.5.4-217-g338f5c0 in docker
Backport?: Not yet backported
Include in Changelog: 1

Examples like this :
$ curl -k -s -u 'root:icinga' -H 'X-HTTP-Method-Override: DELETE' -X POST 'https://localhost:5665/v1/objects/hosts/example.localdomain'
Return :
<h1>Accept header is missing or not set to 'application/json'.</h1>

The parameter -H 'Accept: application/json' of curl is missing.

Changesets

2016-12-06 09:46:47 +00:00 by mfriedrich be935d0

Docs: Fix missing request headers for X-HTTP-Method-Override

fixes #13411
@icinga-migration
Copy link
Author

Updated by jflach on 2016-12-06 08:18:59 +00:00

Feel free to create a patch .

If not we'll get on it sometime

@icinga-migration
Copy link
Author

Updated by tofu00 on 2016-12-06 08:54:49 +00:00

jflach wrote:

Feel free to create a patch .

If not we'll get on it sometime

Yes I will try to take the time.

@icinga-migration
Copy link
Author

Updated by jflach on 2016-12-06 09:21:21 +00:00

  • Status changed from New to Assigned
  • Assigned to set to tofu00
  • Target Version set to 2.7.0

Awesome!

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2016-12-06 09:35:41 +00:00

  • Assigned to changed from tofu00 to mfriedrich
  • Target Version changed from 2.7.0 to 2.6.0

I'm taking over, thanks for the nice catch.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2016-12-06 09:46:01 +00:00

  • Subject changed from Doc - missing API headers in examples to Missing API headers for X-HTTP-Method-Override

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2016-12-06 09:50:03 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset be935d0.

@icinga-migration icinga-migration added bug Something isn't working area/documentation End-user or developer help labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.6.0 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation End-user or developer help bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant