Fix stale conversation sync labels on mobile
This commit is contained in:
@@ -667,6 +667,10 @@ public class BossApiClient {
|
||||
connection.setDoInput(true);
|
||||
connection.setRequestProperty("Accept", "application/json");
|
||||
connection.setRequestProperty("x-boss-native-app", "1");
|
||||
if ("GET".equals(method)) {
|
||||
connection.setRequestProperty("Cache-Control", "no-cache, no-store, max-age=0");
|
||||
connection.setRequestProperty("Pragma", "no-cache");
|
||||
}
|
||||
|
||||
String cookie = getSessionCookie();
|
||||
if (!cookie.isEmpty()) {
|
||||
|
||||
@@ -51,6 +51,8 @@ public class BossApiClientDispatchPlansTest {
|
||||
assertEquals("GET", connection.requestMethodValue);
|
||||
assertEquals(12000, connection.connectTimeoutValue);
|
||||
assertEquals(30000, connection.readTimeoutValue);
|
||||
assertEquals("no-cache, no-store, max-age=0", connection.getRequestProperty("Cache-Control"));
|
||||
assertEquals("no-cache", connection.getRequestProperty("Pragma"));
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -67,6 +69,18 @@ public class BossApiClientDispatchPlansTest {
|
||||
assertEquals(30000, connection.readTimeoutValue);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void getConversationHomeSendsNoCacheHeadersToAvoidStaleMobileFeed() throws Exception {
|
||||
RecordingConnection connection = new RecordingConnection(new URL("https://boss.hyzq.net/api/v1/conversations/home"));
|
||||
RecordingBossApiClient apiClient = new RecordingBossApiClient(connection);
|
||||
|
||||
BossApiClient.ApiResponse response = apiClient.getConversationHome();
|
||||
|
||||
assertEquals(200, response.statusCode);
|
||||
assertEquals("no-cache, no-store, max-age=0", connection.getRequestProperty("Cache-Control"));
|
||||
assertEquals("no-cache", connection.getRequestProperty("Pragma"));
|
||||
}
|
||||
|
||||
@Test
|
||||
public void confirmDispatchPlanWritesApprovedTargetProjectIds() throws Exception {
|
||||
RecordingConnection connection = new RecordingConnection(new URL("https://boss.hyzq.net/api/v1/projects/p1/dispatch-plans/plan-1/confirm"));
|
||||
|
||||
Reference in New Issue
Block a user