HttpGraphQlClient content-type

[marinus-suniram]

The content-type header in HttpGraphQlTransport is hard-code to application/graphql+json. No all graphql servers supports application/graphql+json. Can we make the content-type configurable?

public Mono<GraphQlResponse> execute(GraphQlRequest request) {
	return this.webClient.post()
			.contentType(MediaType.APPLICATION_GRAPHQL)
			.accept(MediaType.APPLICATION_GRAPHQL, MediaType.APPLICATION_JSON)
			.bodyValue(request.toMap())
			.retrieve()
			.bodyToMono(MAP_TYPE)
			.map(ResponseMapGraphQlResponse::new);
}

[bclozel]

The HttpGraphQlTransport has been changed recently. In our published SNAPSHOTs:

spring-graphql/spring-graphql/src/main/java/org/springframework/graphql/client/HttpGraphQlTransport.java

Lines 59 to 71 in 8549060

	private static MediaType initContentType(WebClient webClient) {
	HttpHeaders headers = new HttpHeaders();
	webClient.mutate().defaultHeaders(headers::putAll);
	MediaType contentType = headers.getContentType();
	return (contentType != null ? contentType : MediaType.APPLICATION_JSON);
	}
	@Override
	public Mono<GraphQlResponse> execute(GraphQlRequest request) {
	return this.webClient.post()
	.contentType(this.contentType)
	.accept(MediaType.APPLICATION_JSON, MediaType.APPLICATION_GRAPHQL)

As of #359, you can explicitly set a content type with the client, so this might be a duplicate of this issue.
As of #375, we're now back to using "application/json" as the default.

Is there something else we're missing? Can you try our latest SNAPSHOTs?

[marinus-suniram]

Thanks, I can confirm that the issue is fixed on SNAPSHOT.

[bclozel]

Thanks for trying the SNAPSHOTs!