search

ECMaker / blog

https://blog.ec-maker.com/
0 stars 0 forks source link

Feature/#20/#40 debug #41

Closed u-ecmaker closed 6 months ago

u-ecmaker commented 7 months ago

Feature/#20/#40 debug

マージ先を#20に変更のこと！！！

update

posts/[slug] の useComment を参照している箇所を slug ではなく page_id とすることで解消した。

remaning tasks

netlify[bot] commented 7 months ago

Deploy Preview for blog-ecmaker ready!

Name Link
Latest commit
Latest deploy log https://app.netlify.com/sites/blog-ecmaker/deploys/65cd8a580f4e9844eaa2949b
Deploy Preview https://deploy-preview-41--blog-ecmaker.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

u-ecmaker commented 7 months ago

slug → page_id に戻すことで、以下のエラーは出なくなった。

1a9d254 で解消した。 23e83c3 では以下のエラーが出る。

@notionhq/client warn: request fail {
  code: 'validation_error',
  message: 'query failed validation: query.block_id should be a valid uuid, instead was `"logo"`.'
}
 ⨯ APIResponseError: query failed validation: query.block_id should be a valid uuid, instead was `"logo"`.
    at buildRequestError (C:\Users\yuki\Documents\git\blog\node_modules\@notionhq\client\build\src\errors.js:162:16)
    at Client.request (C:\Users\yuki\Documents\git\blog\node_modules\@notionhq\client\build\src\Client.js:347:54)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async getComments (webpack-internal:///(api)/./src/server/notion/comments.ts:13:22)
    at async handler (webpack-internal:///(api)/./pages/api/notion-blog/comments.ts:14:28) {
  code: 'validation_error',
  status: 400,
  headers: Headers {
    [Symbol(map)]: [Object: null prototype] {
      date: [Array],
      'content-type': [Array],
      'content-length': [Array],
      connection: [Array],
      'x-powered-by': [Array],
      'x-notion-request-id': [Array],
      etag: [Array],
      vary: [Array],
      'cf-cache-status': [Array],
      'set-cookie': [Array],
      server: [Array],
      'cf-ray': [Array]
    }
  },
  body: '{"object":"error","status":400,"code":"validation_error","message":"query failed validation: query.block_id should be a valid uuid, instead was `\\"logo\\"`.","request_id":"2c6943e4-8cb9-4937-9e71-6cc6909cbfcb"}',
  page: '/api/notion-blog/comments'
}
u-ecmaker commented 7 months ago

2abc719 で通った。npm audit fix したらngってことですか。

u-ecmaker commented 7 months ago

↑1個前エラー 146abd0

u-ecmaker commented 7 months ago

revertしたやつじゃないとNGだった。 npm audti fix すると動かなくなってしまったので諦めた。 revertしてからひとつずつ上げていく方針。

npm audit report で 1 high になっている脅威のみに対処 npm i browserify-sign

OK 6b2aa46

u-ecmaker commented 7 months ago

"ogp-parser": "^0.6.0" こいつが脅威だった

u-ecmaker commented 7 months ago

5218d49 脅威は去った and OK but...

24 vulnerabilities (1 low, 14 moderate, 9 critical)

u-ecmaker commented 7 months ago

ここでエラー d1a2213 ここは通った b68be10

これ、すなわち、nextは固定すべしってことかな

u-ecmaker commented 7 months ago

5b30cd6 OK. packageはここまでとする。 image

next 0.9.9 - 13.5.4-canary.11

next のverはupdateできなそう。 https://answers.netlify.com/t/nextjs-error-500-after-deploy-when-fetching-data-on-api-route/107800/10

postcss <8.4.31

No fix available どうしようもない(?) image

u-ecmaker commented 6 months ago

36 コメントのエラーは出るが、

image

この辺のエラーは消えた image

posts/[slug] の useComment を参照している箇所を slug ではなく page_id とすることで解消した。