@quintype/framework 7.34.7-handle-500-issue.1 → 7.34.7-handle-500-issue.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (6) hide show
  1. package/package.json +1 -1
  2. package/server/handlers/cdn-caching.js +1 -1
  3. package/server/handlers/isomorphic-handler.js +0 -2
  4. package/test/integration/custom-route-handler-test.js +13 -44
  5. package/test/integration/isomorphic-handler-test.js +6 -9
  6. package/test/integration/url-redirect-test.js +10 -40
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@quintype/framework",
3
- "version": "7.34.7-handle-500-issue.1",
3
+ "version": "7.34.7-handle-500-issue.2",
4
4
  "description": "Libraries to help build Quintype Node.js apps",
5
5
  "main": "index.js",
6
6
  "engines": {
package/server/handlers/cdn-caching.js CHANGED
@@ -42,7 +42,7 @@ exports.addCacheHeadersToResult = function addCacheHeadersToResult({
42
42
  } else {
43
43
  res.setHeader(
44
44
  "Cache-Control",
45
- `public,max-age=${maxAge},s-maxage=${sMaxAge},stale-while-revalidate=1000,stale-if-error=${STALE_IF_ERROR_CACHE_DURATION}`
45
+ `public,max-age=${maxAge},s-maxage=120,stale-while-revalidate=120,stale-if-error=120`
46
46
  );
47
47
  }
48
48
 
package/server/handlers/isomorphic-handler.js CHANGED
@@ -488,8 +488,6 @@ exports.handleIsomorphicRoute = function handleIsomorphicRoute(
488
488
 
489
489
  // For server errors (>=500),send statuss
490
490
  if (statusCode >= 500) {
491
- res.status(statusCode);
492
- res.setHeader("Content-Type", "text/html; charset=utf-8");
493
491
  return res.sendStatus(statusCode);
494
492
  }
495
493
 
package/test/integration/custom-route-handler-test.js CHANGED
@@ -48,8 +48,7 @@ function getClientStub(hostname) {
48
48
  page: {
49
49
  id: 103,
50
50
  title: "Testing",
51
- content:
52
- "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
51
+ content: "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
53
52
  metadata: { header: true, footer: false },
54
53
  type: "static-page",
55
54
  "status-code": 200,
@@ -60,8 +59,7 @@ function getClientStub(hostname) {
60
59
  page: {
61
60
  id: 104,
62
61
  title: "Testing",
63
- content:
64
- "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
62
+ content: "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
65
63
  metadata: { header: false, footer: false },
66
64
  type: "static-page",
67
65
  "status-code": 200,
@@ -72,8 +70,7 @@ function getClientStub(hostname) {
72
70
  page: {
73
71
  id: 105,
74
72
  title: "Testing mime type",
75
- content:
76
- "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
73
+ content: "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
77
74
  metadata: {
78
75
  header: false,
79
76
  footer: false,
@@ -104,9 +101,7 @@ function createApp(loadData, routes, opts = {}) {
104
101
  generateRoutes: () => routes,
105
102
  loadData,
106
103
  renderLayout: (res, { contentTemplate, store }) =>
107
- res.send(
108
- JSON.stringify({ contentTemplate, store: store.getState() })
109
- ),
104
+ res.send(JSON.stringify({ contentTemplate, store: store.getState() })),
110
105
  handleNotFound: false,
111
106
  publisherConfig: {},
112
107
  },
@@ -126,10 +121,7 @@ describe("Custom Route Handler", function () {
126
121
  supertest(app)
127
122
  .get("/moved-permanently")
128
123
  .expect("Location", "/permanent-location")
129
- .expect(
130
- "Cache-Control",
131
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
132
- )
124
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
133
125
  .expect("Vary", /Accept\-Encoding/)
134
126
  .expect("Surrogate-Key", "sp/42/101")
135
127
  .expect("Cache-Tag", "sp/42/101")
@@ -144,10 +136,7 @@ describe("Custom Route Handler", function () {
144
136
  supertest(app)
145
137
  .get("/moved-temporarily")
146
138
  .expect("Location", "/temporary-location")
147
- .expect(
148
- "Cache-Control",
149
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
150
- )
139
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
151
140
  .expect("Vary", /Accept\-Encoding/)
152
141
  .expect("Surrogate-Key", "sp/42/102")
153
142
  .expect("Cache-Tag", "sp/42/102")
@@ -163,10 +152,7 @@ describe("Custom Route Handler", function () {
163
152
  supertest(app)
164
153
  .get("/static-with-header-footer")
165
154
  .expect("Content-Type", /html/)
166
- .expect(
167
- "Cache-Control",
168
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
169
- )
155
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
170
156
  .expect("Vary", "Accept-Encoding")
171
157
  .expect("Surrogate-Key", "sp/42/103")
172
158
  .expect("Cache-Tag", "sp/42/103")
@@ -188,19 +174,13 @@ describe("Custom Route Handler", function () {
188
174
  supertest(app)
189
175
  .get("/static-without-header-footer")
190
176
  .expect("Content-Type", /html/)
191
- .expect(
192
- "Cache-Control",
193
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
194
- )
177
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
195
178
  .expect("Vary", "Accept-Encoding")
196
179
  .expect("Surrogate-Key", "sp/42/104")
197
180
  .expect("Cache-Tag", "sp/42/104")
198
181
  .expect(200)
199
182
  .then((res) => {
200
- assert.equal(
201
- "<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>",
202
- res.text
203
- );
183
+ assert.equal("<html><head><title>Test</title></head><body><h1>Heading</h1></body></html>", res.text);
204
184
  })
205
185
  .then(done);
206
186
  });
@@ -233,9 +213,7 @@ describe("Custom Route Handler", function () {
233
213
  it("Store reads config and data from data-loader response", function (done) {
234
214
  const app = createApp(
235
215
  (pageType, params, config, client, { host, next }) =>
236
- pageType === "custom-static-page"
237
- ? Promise.resolve({ data: { navigationMenu: [] }, config: {} })
238
- : next(),
216
+ pageType === "custom-static-page" ? Promise.resolve({ data: { navigationMenu: [] }, config: {} }) : next(),
239
217
  [{ pageType: "story-page", path: "/*" }]
240
218
  );
241
219
  supertest(app)
@@ -258,10 +236,7 @@ describe("Custom Route Handler", function () {
258
236
  supertest(app)
259
237
  .get("/moved-absolute")
260
238
  .expect("Location", "https://www.google.com")
261
- .expect(
262
- "Cache-Control",
263
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
264
- )
239
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
265
240
  .expect("Vary", /Accept\-Encoding/)
266
241
  .expect("Surrogate-Key", "sp/42/105")
267
242
  .expect("Cache-Tag", "sp/42/105")
@@ -275,10 +250,7 @@ describe("Custom Route Handler", function () {
275
250
  );
276
251
  supertest(app)
277
252
  .get("/static-with-mime-type")
278
- .expect(
279
- "Cache-Control",
280
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
281
- )
253
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
282
254
  .expect("Vary", /Accept\-Encoding/)
283
255
  .expect("Surrogate-Key", "sp/42/105")
284
256
  .expect("Cache-Tag", "sp/42/105")
@@ -293,10 +265,7 @@ describe("Custom Route Handler", function () {
293
265
  );
294
266
  supertest(app)
295
267
  .get("/static-without-header-footer")
296
- .expect(
297
- "Cache-Control",
298
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
299
- )
268
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
300
269
  .expect("Vary", "Accept-Encoding")
301
270
  .expect("Surrogate-Key", "sp/42/104")
302
271
  .expect("Cache-Tag", "sp/42/104")
package/test/integration/isomorphic-handler-test.js CHANGED
@@ -200,7 +200,7 @@ describe("Isomorphic Handler", function () {
200
200
  supertest(app)
201
201
  .get("/")
202
202
  .expect("Content-Type", /html/)
203
- .expect("Cache-Control", "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400")
203
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
204
204
  .expect("Vary", "Accept-Encoding")
205
205
  .expect("Surrogate-Key", "foo bar")
206
206
  .expect("Cache-Tag", "foo,bar")
@@ -503,7 +503,7 @@ describe("Isomorphic Handler", function () {
503
503
  .then((res) => {
504
504
  const cacheControl = res.header["cache-control"];
505
505
  const cacheTag = res.header["cache-tag"];
506
- assert.equal(cacheControl, "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400");
506
+ assert.equal(cacheControl, "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120");
507
507
  assert.equal(cacheTag, "c/1/abcdefgh");
508
508
  })
509
509
  .then(done);
@@ -631,7 +631,7 @@ describe("Isomorphic Handler", function () {
631
631
  const cacheControl = res.header["cache-control"];
632
632
  const edgeCacheTag = res.header["edge-cache-tag"];
633
633
  const contentSecurityPolicy = res.header["content-security-policy"];
634
- assert.equal(cacheControl, "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400");
634
+ assert.equal(cacheControl, "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120");
635
635
  assert.equal(edgeCacheTag, "c/1/abcdefgh");
636
636
  assert.equal(
637
637
  contentSecurityPolicy,
@@ -670,7 +670,7 @@ describe("Isomorphic Handler", function () {
670
670
  const cacheControl = res.header["cache-control"];
671
671
  const edgeCacheTag = res.header["edge-cache-tag"];
672
672
  const contentSecurityPolicy = res.header["content-security-policy"];
673
- assert.equal(cacheControl, "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400");
673
+ assert.equal(cacheControl, "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120");
674
674
  assert.equal(edgeCacheTag, "c/1/abcdefgh");
675
675
  assert.equal(
676
676
  contentSecurityPolicy,
@@ -706,10 +706,7 @@ describe("Isomorphic Handler", function () {
706
706
  .expect(200)
707
707
  .then((res) => {
708
708
  const cacheControl = res.header["cache-control"];
709
- assert.equal(
710
- cacheControl,
711
- "public,max-age=15,s-maxage=1800,stale-while-revalidate=1000,stale-if-error=14400"
712
- );
709
+ assert.equal(cacheControl, "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120");
713
710
  })
714
711
  .then(done);
715
712
  });
@@ -730,7 +727,7 @@ describe("Isomorphic Handler", function () {
730
727
  .expect(200)
731
728
  .then((res) => {
732
729
  const cacheControl = res.header["cache-control"];
733
- assert.equal(cacheControl, "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400");
730
+ assert.equal(cacheControl, "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120");
734
731
  })
735
732
  .then(done);
736
733
  });
package/test/integration/url-redirect-test.js CHANGED
@@ -508,10 +508,7 @@ describe("Redirect Routes Handler", function () {
508
508
  redirectUrls,
509
509
  }
510
510
  );
511
- supertest(app)
512
- .get("/moved-permanently-1")
513
- .expect("Location", "/permanent-location-1")
514
- .expect(301, done);
511
+ supertest(app).get("/moved-permanently-1").expect("Location", "/permanent-location-1").expect(301, done);
515
512
  });
516
513
 
517
514
  it("Redirects all the urls with status code to 302 if redirectUrls is present", function (done) {
@@ -528,10 +525,7 @@ describe("Redirect Routes Handler", function () {
528
525
  ],
529
526
  }
530
527
  );
531
- supertest(app)
532
- .get("/moved-temporarily-1")
533
- .expect("Location", "/temporarily-location-1")
534
- .expect(302, done);
528
+ supertest(app).get("/moved-temporarily-1").expect("Location", "/temporarily-location-1").expect(302, done);
535
529
  });
536
530
 
537
531
  it("Renders homepage when redirect urls are present", function (done) {
@@ -555,10 +549,7 @@ describe("Redirect Routes Handler", function () {
555
549
  .expect(200)
556
550
  .then((res) => {
557
551
  const response = JSON.parse(res.text);
558
- assert.equal(
559
- '<div data-page-type="home-page">foobar</div>',
560
- response.content
561
- );
552
+ assert.equal('<div data-page-type="home-page">foobar</div>', response.content);
562
553
  assert.equal("foobar", response.store.qt.data.text);
563
554
  assert.equal("127.0.0.1", response.store.qt.data.host);
564
555
  assert.equal("home-page", response.store.qt.pageType);
@@ -572,10 +563,7 @@ describe("Redirect Routes Handler", function () {
572
563
  [{ pageType: "story-page", path: "/*" }],
573
564
  { redirectUrls: getRedirectUrl }
574
565
  );
575
- supertest(app)
576
- .get("/moved-permanently-1")
577
- .expect("Location", "/permanent-location-1")
578
- .expect(301, done);
566
+ supertest(app).get("/moved-permanently-1").expect("Location", "/permanent-location-1").expect(301, done);
579
567
  });
580
568
 
581
569
  it("Should handle Route Parameters redirects properly", function (done) {
@@ -584,10 +572,7 @@ describe("Redirect Routes Handler", function () {
584
572
  [{ pageType: "story-page", path: "/*" }],
585
573
  { redirectUrls: getRedirectUrl }
586
574
  );
587
- supertest(app)
588
- .get("/india/news/some-slug")
589
- .expect("Location", "/india/some-slug")
590
- .expect(301, done);
575
+ supertest(app).get("/india/news/some-slug").expect("Location", "/india/some-slug").expect(301, done);
591
576
  });
592
577
 
593
578
  it("Should handle multiple Route Parameters redirects properly", function (done) {
@@ -596,10 +581,7 @@ describe("Redirect Routes Handler", function () {
596
581
  [{ pageType: "story-page", path: "/*" }],
597
582
  { redirectUrls: getRedirectUrl }
598
583
  );
599
- supertest(app)
600
- .get("/india/foo/some-slug/slug")
601
- .expect("Location", "/india/some-slug")
602
- .expect(302, done);
584
+ supertest(app).get("/india/foo/some-slug/slug").expect("Location", "/india/some-slug").expect(302, done);
603
585
  });
604
586
 
605
587
  it("Should handle interchanging route parameter redirects properly", function (done) {
@@ -608,10 +590,7 @@ describe("Redirect Routes Handler", function () {
608
590
  [{ pageType: "story-page", path: "/*" }],
609
591
  { redirectUrls: getRedirectUrl }
610
592
  );
611
- supertest(app)
612
- .get("/foo/something/bar")
613
- .expect("Location", "/bar/something/foo")
614
- .expect(301, done);
593
+ supertest(app).get("/foo/something/bar").expect("Location", "/bar/something/foo").expect(301, done);
615
594
  });
616
595
 
617
596
  it("Should handle external redirects", function (done) {
@@ -620,10 +599,7 @@ describe("Redirect Routes Handler", function () {
620
599
  [{ pageType: "story-page", path: "/*" }],
621
600
  { redirectUrls: getRedirectUrl }
622
601
  );
623
- supertest(app)
624
- .get("/something/new")
625
- .expect("Location", "https://www.google.com/something")
626
- .expect(301, done);
602
+ supertest(app).get("/something/new").expect("Location", "https://www.google.com/something").expect(301, done);
627
603
  });
628
604
 
629
605
  it("Should handle external redirects with params", function (done) {
@@ -632,10 +608,7 @@ describe("Redirect Routes Handler", function () {
632
608
  [{ pageType: "story-page", path: "/*" }],
633
609
  { redirectUrls: getRedirectUrl }
634
610
  );
635
- supertest(app)
636
- .get("/something/new1")
637
- .expect("Location", "https://www.google.com/new1")
638
- .expect(301, done);
611
+ supertest(app).get("/something/new1").expect("Location", "https://www.google.com/new1").expect(301, done);
639
612
  });
640
613
 
641
614
  it("Should not crash if redirectUrls is not present", function (done) {
@@ -646,10 +619,7 @@ describe("Redirect Routes Handler", function () {
646
619
  supertest(app)
647
620
  .get("/moved-temporarily-1")
648
621
  .expect("Location", "/temporarily-location-1")
649
- .expect(
650
- "Cache-Control",
651
- "public,max-age=15,s-maxage=900,stale-while-revalidate=1000,stale-if-error=14400"
652
- )
622
+ .expect("Cache-Control", "public,max-age=15,s-maxage=120,stale-while-revalidate=120,stale-if-error=120")
653
623
  .expect("Vary", /Accept\-Encoding/)
654
624
  .expect("Surrogate-Key", "sp/42/102")
655
625
  .expect("Cache-Tag", "sp/42/102")