Skip to content

Fix: fix mysterious digest error bug #353

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Aug 29, 2024
Merged

Fix: fix mysterious digest error bug #353

merged 1 commit into from
Aug 29, 2024

Conversation

@StrongMonkey
Copy link
Contributor

@StrongMonkey StrongMonkey commented Aug 29, 2024 •
edited
Loading

This should fix the bug

2024-08-29T05:47:14.205Z [server] [ERROR]  ⨯ unhandledRejection: TypeError: Cannot read properties of undefined (reading 'digest')
    at /Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:36:17710
    at AsyncLocalStorage.run (node:async_hooks:346:14)
    at eL (/Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:35:273726)
    at /Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:35:269554
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2024-08-29T05:47:14.206Z [server] [ERROR]  ⨯ unhandledRejection: TypeError: Cannot read properties of undefined (reading 'digest')
    at /Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:36:17710
    at AsyncLocalStorage.run (node:async_hooks:346:14)
    at eL (/Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:35:273726)
    at /Users/daishan/work/ui/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:35:269554
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

This is because we are sending an empty tool to run stringify.

#257

@StrongMonkey
Fix: fix mysterious digest error bug
88a0271 
Signed-off-by: Daishan Peng <daishan@acorn.io>
@StrongMonkey StrongMonkey merged commit 4d8baa6 into gptscript-ai:main Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cjellick cjellick cjellick approved these changes

@thedadams thedadams Awaiting requested review from thedadams

@njhale njhale Awaiting requested review from njhale

@g-linville g-linville Awaiting requested review from g-linville

@ryanhopperlowe ryanhopperlowe Awaiting requested review from ryanhopperlowe

+1 more reviewer

@iwilltry42 iwilltry42 iwilltry42 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@StrongMonkey @cjellick @iwilltry42