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

Duplicate content type added for sendMail POST request -tst #533

Closed
MIchaelMainer opened this issue May 7, 2020 · 0 comments · Fixed by #537
Closed

Duplicate content type added for sendMail POST request -tst #533

MIchaelMainer opened this issue May 7, 2020 · 0 comments · Fixed by #537
Assignees
Labels
type:bug Something isn't working

Comments

@MIchaelMainer
Copy link
Contributor

Describe the bug
Graph Explorer -tst is adding duplicate content-type value to the Content-Type header. The sample has a default value content-type header value, and then it appears that Graph Explorer is adding a second header value.

To Reproduce
Steps to reproduce the behavior:

  1. Choose the **send an email ** sample.
  2. Click on Run query

Expected behavior
The request should be successful. The response should not show:

{
    "error": {
        "code": "RequestBodyRead",
        "message": "The specified content type 'application/json, application/json' contains either no media type or more than one media type, which is not allowed. You must specify exactly one media type as the content type.",
        "innerError": {
            "request-id": "e42b6e3d-aec0-4e66-8146-9237bfd497d3",
            "date": "2020-05-07T23:22:42"
        }
    }
}

Additional context
If you remove the content-type header from the request headers, GE adds a content-type header. This appears to be a problem for all POST requests. It may be an issue for other HTTP methods.

@MIchaelMainer MIchaelMainer added the type:bug Something isn't working label May 7, 2020
@Shjokie Shjokie added this to To do in Graph Explorer V4 via automation May 8, 2020
@thewahome thewahome linked a pull request May 11, 2020 that will close this issue
@thewahome thewahome moved this from To do to Done in Graph Explorer V4 May 11, 2020
@thewahome thewahome self-assigned this May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants