René's URL Explorer Experiment


Title: test: remove timer from test-http-1.0 by santigimeno · Pull Request #5129 · nodejs/node · GitHub

Open Graph Title: test: remove timer from test-http-1.0 by santigimeno · Pull Request #5129 · nodejs/node

X Title: test: remove timer from test-http-1.0 by santigimeno · Pull Request #5129 · nodejs/node

Description: Node.js JavaScript runtime ✨🐢🚀✨. Contribute to nodejs/node development by creating an account on GitHub.

Open Graph Description: It's possible that the end event is emitted after the timeout fires causing the test to fail. Just remove the timer. If for some reason the end would never fire, the test will fail with a timeo...

X Description: It's possible that the end event is emitted after the timeout fires causing the test to fail. Just remove the timer. If for some reason the end would never fire, the test will fail with a t...

Opengraph URL: https://github.com/nodejs/node/pull/5129

X: @github

direct link

Domain: github.com

route-pattern/_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format)
route-controllervoltron_pull_requests_fragments
route-actionpull_request_layout
fetch-noncev2:860f8d33-fc8a-e42a-aece-364ae2de9897
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idB7DA:1368EB:1F0D060:2ACEFF8:6A4ECCDE
html-safe-nonce3fb95930216c77c32113d58339d8f446a1d576bb1b98b043efbf69528523067c
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCN0RBOjEzNjhFQjoxRjBEMDYwOjJBQ0VGRjg6NkE0RUNDREUiLCJ2aXNpdG9yX2lkIjoiMjM4NDE0OTQwODc4ODY5NjI4NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac39c6b038d895df1baf885957772233d4055ebcaa234c854cb77edb4f1ad7fddd
hovercard-subject-tagpull_request:58542661
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/pull_requests_fragments/pull_request_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/nodejs/node/5129/pull_request_layout
twitter:imagehttps://opengraph.githubassets.com/bcef1c82c9cdc82f9b88e37f3d0183afb7520f411f890c35ad67a5dab391cbda/nodejs/node/pull/5129
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/bcef1c82c9cdc82f9b88e37f3d0183afb7520f411f890c35ad67a5dab391cbda/nodejs/node/pull/5129
og:image:altIt's possible that the end event is emitted after the timeout fires causing the test to fail. Just remove the timer. If for some reason the end would never fire, the test will fail with a timeo...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamesantigimeno
hostnamegithub.com
expected-hostnamegithub.com
None41b6ab3ba6d20a71766ac245b5a4a94c6fc672a9cd4da7d44c1b33ab8bf6a21c
turbo-cache-controlno-cache
go-importgithub.com/nodejs/node git https://github.com/nodejs/node.git
octolytics-dimension-user_id9950313
octolytics-dimension-user_loginnodejs
octolytics-dimension-repository_id27193779
octolytics-dimension-repository_nwonodejs/node
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id27193779
octolytics-dimension-repository_network_root_nwonodejs/node
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
releasee6a744804e8e70f97b4d5a18a94dcc63db22f97a
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/nodejs/node/pull/5129#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F5129
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/open-source/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/open-source/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F5129
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=nodejs%2Fnode
Reloadhttps://github.com/nodejs/node/pull/5129
Reloadhttps://github.com/nodejs/node/pull/5129
Reloadhttps://github.com/nodejs/node/pull/5129
Please reload this pagehttps://github.com/nodejs/node/pull/5129
nodejs https://github.com/nodejs
nodehttps://github.com/nodejs/node
Please reload this pagehttps://github.com/nodejs/node/pull/5129
Notifications https://github.com/login?return_to=%2Fnodejs%2Fnode
Fork 36k https://github.com/login?return_to=%2Fnodejs%2Fnode
Star 118k https://github.com/login?return_to=%2Fnodejs%2Fnode
Code https://github.com/nodejs/node
Issues 1.3k https://github.com/nodejs/node/issues
Pull requests 965 https://github.com/nodejs/node/pulls
Actions https://github.com/nodejs/node/actions
Projects https://github.com/nodejs/node/projects
Security and quality 0 https://github.com/nodejs/node/security
Insights https://github.com/nodejs/node/pulse
Code https://github.com/nodejs/node
Issues https://github.com/nodejs/node/issues
Pull requests https://github.com/nodejs/node/pulls
Actions https://github.com/nodejs/node/actions
Projects https://github.com/nodejs/node/projects
Security and quality https://github.com/nodejs/node/security
Insights https://github.com/nodejs/node/pulse
santigimenohttps://github.com/santigimeno
nodejs:masterhttps://github.com/nodejs/node/tree/master
santigimeno:test-http-1.0https://github.com/santigimeno/node/tree/test-http-1.0
Conversationhttps://github.com/nodejs/node/pull/5129
Commits1 (1)https://github.com/nodejs/node/pull/5129/commits
Checkshttps://github.com/nodejs/node/pull/5129/checks
Files changedhttps://github.com/nodejs/node/pull/5129/files
test: remove timer from test-http-1.0https://github.com/nodejs/node/pull/5129#top
santigimenohttps://github.com/santigimeno
nodejs:masterhttps://github.com/nodejs/node/tree/master
santigimeno:test-http-1.0https://github.com/santigimeno/node/tree/test-http-1.0
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
Feb 7, 2016https://github.com/nodejs/node/pull/5129#issue-131952254
#4600 (comment)https://github.com/nodejs/node/pull/4600#issuecomment-180388736
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/Trott
Trotthttps://github.com/Trott
Feb 7, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181033602
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/Trott
Trotthttps://github.com/Trott
Feb 7, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181034264
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/r-52
r-52https://github.com/r-52
http https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Ahttp
test https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Atest
lts-watch-v4.x https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Alts-watch-v4.x
Feb 7, 2016https://github.com/nodejs/node/pull/5129#event-542205397
https://github.com/jasnell
jasnellhttps://github.com/jasnell
Feb 7, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181114436
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/santigimeno
test: remove timer from test-http-1.0https://github.com/nodejs/node/pull/5129/commits/d77f3058eca27fb03956a43b00f82e15b48eaed5
d77f305https://github.com/nodejs/node/pull/5129/commits/d77f3058eca27fb03956a43b00f82e15b48eaed5
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
force-pushedhttps://github.com/nodejs/node/compare/5233710ab92eb38df775721d20453655b69133ca..d77f3058eca27fb03956a43b00f82e15b48eaed5
5233710https://github.com/nodejs/node/commit/5233710ab92eb38df775721d20453655b69133ca
d77f305https://github.com/nodejs/node/commit/d77f3058eca27fb03956a43b00f82e15b48eaed5
Compare https://github.com/nodejs/node/compare/5233710ab92eb38df775721d20453655b69133ca..d77f3058eca27fb03956a43b00f82e15b48eaed5
February 8, 2016 18:55https://github.com/nodejs/node/pull/5129#event-543351291
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
Feb 8, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181519283
@Trotthttps://github.com/Trott
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
Feb 8, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181519933
@Trotthttps://github.com/Trott
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/Trott
Trotthttps://github.com/Trott
Feb 8, 2016https://github.com/nodejs/node/pull/5129#issuecomment-181571132
@santigimenohttps://github.com/santigimeno
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
Feb 16, 2016https://github.com/nodejs/node/pull/5129#issuecomment-184833212
@Trotthttps://github.com/Trott
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/Trott
Trotthttps://github.com/Trott
Feb 16, 2016https://github.com/nodejs/node/pull/5129#issuecomment-184841369
https://ci.nodejs.org/job/node-test-pull-request/1679/https://ci.nodejs.org/job/node-test-pull-request/1679/
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/jasnell
jasnellhttps://github.com/jasnell
Feb 17, 2016https://github.com/nodejs/node/pull/5129#issuecomment-185286341
https://ci.nodejs.org/job/node-test-pull-request/1685/https://ci.nodejs.org/job/node-test-pull-request/1685/
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/santigimeno
santigimenohttps://github.com/santigimeno
Feb 24, 2016 https://github.com/nodejs/node/pull/5129#ref-issue-136228278
Review the use of timeouts nodejs/testing#19 https://github.com/nodejs/testing/issues/19
https://github.com/jasnell
jasnellhttps://github.com/jasnell
Mar 21, 2016https://github.com/nodejs/node/pull/5129#issuecomment-199518533
@Trotthttps://github.com/Trott
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/Trott
Trotthttps://github.com/Trott
Mar 21, 2016https://github.com/nodejs/node/pull/5129#issuecomment-199539559
@jasnellhttps://github.com/jasnell
Please reload this pagehttps://github.com/nodejs/node/pull/5129
jasnellhttps://github.com/jasnell
Mar 22, 2016 https://github.com/nodejs/node/pull/5129#ref-commit-6a3d38f
https://github.com/santigimeno
https://github.com/jasnell
test: remove timer from test-http-1.0https://github.com/nodejs/node/commit/6a3d38f00f116cbe51cd006ee558ca773522e6f5
6a3d38fhttps://github.com/nodejs/node/commit/6a3d38f00f116cbe51cd006ee558ca773522e6f5
#5129https://github.com/nodejs/node/pull/5129
https://github.com/jasnell
jasnellhttps://github.com/jasnell
Mar 22, 2016https://github.com/nodejs/node/pull/5129#issuecomment-199550394
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/jasnell
jasnellhttps://github.com/jasnell
Mar 22, 2016https://github.com/nodejs/node/pull/5129#event-597973977
https://github.com/Fishrock123
Fishrock123https://github.com/Fishrock123
Mar 22, 2016 https://github.com/nodejs/node/pull/5129#ref-pullrequest-142415910
v5.9.1 Release Proposal #5831 https://github.com/nodejs/node/pull/5831
Fishrock123https://github.com/Fishrock123
Mar 22, 2016 https://github.com/nodejs/node/pull/5129#ref-commit-7c2c7b0
https://github.com/santigimeno
https://github.com/Fishrock123
test: remove timer from test-http-1.0https://github.com/nodejs/node/commit/7c2c7b05770fa71288efe8abf6238506d9fe27fb
7c2c7b0https://github.com/nodejs/node/commit/7c2c7b05770fa71288efe8abf6238506d9fe27fb
#5129https://github.com/nodejs/node/pull/5129
MylesBorinshttps://github.com/MylesBorins
Mar 30, 2016 https://github.com/nodejs/node/pull/5129#ref-commit-1cc070b
https://github.com/santigimeno
test: remove timer from test-http-1.0https://github.com/nodejs/node/commit/1cc070b4cc32a810a9e75ba5e8a748ffcc7e562f
1cc070bhttps://github.com/nodejs/node/commit/1cc070b4cc32a810a9e75ba5e8a748ffcc7e562f
#5129https://github.com/nodejs/node/pull/5129
https://github.com/MylesBorins
MylesBorinshttps://github.com/MylesBorins
land-on-v4.x https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Aland-on-v4.x
lts-watch-v4.x https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Alts-watch-v4.x
Mar 30, 2016https://github.com/nodejs/node/pull/5129#event-608670123
MylesBorinshttps://github.com/MylesBorins
Mar 30, 2016 https://github.com/nodejs/node/pull/5129#ref-commit-f209eff
https://github.com/santigimeno
test: remove timer from test-http-1.0https://github.com/nodejs/node/commit/f209effe8b9119243a11134b91bc6410b9fda8b5
f209effhttps://github.com/nodejs/node/commit/f209effe8b9119243a11134b91bc6410b9fda8b5
#5129https://github.com/nodejs/node/pull/5129
[Snyk] Fix for 114 vulnerabilities aliscco/alisco-node#338 https://github.com/aliscco/alisco-node/pull/338
[Snyk] Fix for 4 vulnerabilities aliscco/alisco-node#349 https://github.com/aliscco/alisco-node/pull/349
https://github.co/hiddenchars
https://github.com/nodejs/node/pull/{{ revealButtonHref }}
Sign up for freehttps://github.com/join?source=comment-repo
Sign in to commenthttps://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fpull%2F5129
http https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Ahttp
test https://github.com/nodejs/node/issues?q=state%3Aopen%20label%3Atest
Please reload this pagehttps://github.com/nodejs/node/pull/5129
https://github.com/santigimeno
https://github.com/Trott
https://github.com/jasnell
https://github.com/MylesBorins
https://github.com/r-52
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.