search

spring-projects / spring-graphql

Spring Integration for GraphQL
https://spring.io/projects/spring-graphql
Apache License 2.0
1.5k stars 297 forks source link

Add Keep-alive message type. #967

Closed mehdizebhi closed 2 months ago

mehdizebhi commented 2 months ago

In these changes, I added a "KA" instance to GraphQlWebSocketMessageType to support this type. Likewise, on the WebSocket client side, in the WebSocketGraphQlTransport class and the handle method, if the type "KA" is received from the server side, the IllegalStateException error will not be thrown, and there is no need for any reaction from the client side. Like receiving a pong message.

pivotal-cla commented 2 months ago

@mehdizebhi Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

rstoyanchev commented 2 months ago

Closing as per https://github.com/spring-projects/spring-graphql/issues/965#issuecomment-2114954703.