Skip to content

fix: disable example /get bug. 'require is not defined' #5069

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
Closed
SoraY677 wants to merge 3 commits into from
Closed

fix: disable example /get bug. 'require is not defined' #5069

SoraY677 wants to merge 3 commits into from

Conversation

@SoraY677
Copy link

@SoraY677 SoraY677 commented Oct 9, 2022 •
edited
Loading

Instructions

detail: #5068

when I try to run examples and acccess localhost:3000/get, displayed "Network Error".
the console executing npm run examples show

ReferenceError: require is not defined
   at Server.<anonymous> (file:///C:/Users/FirstName%20Lastname/Desktop/axios/examples/server.js:131:7) 
   at Server.emit (node:events:390:28)
   at parserOnIncoming (node:_http_server:951:12)
   at HTTPParser.parserOnHeadersComplete (node:_http_common:128:17)

this problems was happened by using require-function (set on package.json type: module,
can`t use require)

so, change to use dynamic-import.

@jasonsaayman
Copy link
Member

jasonsaayman commented Oct 30, 2022

This will be solved in the latest release closing for now

@Pulset
Copy link

Pulset commented Feb 7, 2023

This will be solved in the latest release closing for now

This code is not merged in v1.3.2.The problem still exists

@naronchen
Copy link
Contributor

naronchen commented Sep 10, 2023 •
edited
Loading

yh I'm encountering the same error

@kuroino721 kuroino721 mentioned this pull request Aug 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@SoraY677 @jasonsaayman @Pulset @naronchen