Skip to content

Commit cc60866

Browse files
committed
chore: mark methods as deprecated
1 parent a7ae98d commit cc60866

File tree

2 files changed

+35
-1
lines changed

2 files changed

+35
-1
lines changed

graphql-java-servlet/src/main/java/graphql/kickstart/servlet/context/DefaultGraphQLServletContext.java

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import java.util.List;
66
import java.util.Map;
77
import java.util.stream.Collectors;
8-
import javax.security.auth.Subject;
98
import javax.servlet.http.HttpServletRequest;
109
import javax.servlet.http.HttpServletResponse;
1110
import javax.servlet.http.Part;
@@ -35,25 +34,49 @@ public static Builder createServletContext() {
3534
return new Builder(new DataLoaderRegistry());
3635
}
3736

37+
/**
38+
* @deprecated Use
39+
* <tt>dataFetchingEnvironment.getGraphQlContext().get(HttpServletRequest.class)</tt> instead.
40+
* Since 13.0.0
41+
*/
3842
@Override
43+
@Deprecated
3944
public HttpServletRequest getHttpServletRequest() {
4045
return httpServletRequest;
4146
}
4247

48+
/**
49+
* @deprecated Use
50+
* <tt>dataFetchingEnvironment.getGraphQlContext().get(HttpServletResponse.class)</tt>
51+
* instead. Since 13.0.0
52+
*/
4353
@Override
54+
@Deprecated
4455
public HttpServletResponse getHttpServletResponse() {
4556
return httpServletResponse;
4657
}
4758

59+
/**
60+
* @deprecated Use
61+
* <tt>dataFetchingEnvironment.getGraphQlContext().get(HttpServletRequest.class)</tt> instead
62+
* to get the request and retrieve the file parts yourself. Since 13.0.0
63+
*/
4864
@Override
65+
@Deprecated
4966
@SneakyThrows
5067
public List<Part> getFileParts() {
5168
return httpServletRequest.getParts().stream()
5269
.filter(part -> part.getContentType() != null)
5370
.collect(Collectors.toList());
5471
}
5572

73+
/**
74+
* @deprecated Use
75+
* <tt>dataFetchingEnvironment.getGraphQlContext().get(HttpServletRequest.class)</tt> instead
76+
* to get the request and retrieve the parts yourself. Since 13.0.0
77+
*/
5678
@Override
79+
@Deprecated
5780
@SneakyThrows
5881
public Map<String, List<Part>> getParts() {
5982
return httpServletRequest.getParts().stream().collect(Collectors.groupingBy(Part::getName));

graphql-java-servlet/src/main/java/graphql/kickstart/servlet/context/DefaultGraphQLWebSocketContext.java

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,23 @@ public static Builder createWebSocketContext() {
2828
return new Builder(new DataLoaderRegistry());
2929
}
3030

31+
/**
32+
* @deprecated Use <tt>dataFetchingEnvironment.getGraphQlContext().get(Session.class)</tt>
33+
* instead. Since 13.0.0
34+
*/
3135
@Override
36+
@Deprecated
3237
public Session getSession() {
3338
return session;
3439
}
3540

41+
/**
42+
* @deprecated Use
43+
* <tt>dataFetchingEnvironment.getGraphQlContext().get(HandshakeRequest.class)</tt> instead.
44+
* Since 13.0.0
45+
*/
3646
@Override
47+
@Deprecated
3748
public HandshakeRequest getHandshakeRequest() {
3849
return handshakeRequest;
3950
}

0 commit comments

Comments
 (0)