Skip to content
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

Improve readyState behavior to more reliably report connection state #14727

Closed
2 tasks done
alexbevi opened this issue Jul 5, 2024 · 1 comment
Closed
2 tasks done

Improve readyState behavior to more reliably report connection state #14727

alexbevi opened this issue Jul 5, 2024 · 1 comment
Labels
enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature new feature This change adds new functionality, like a new method or class
Milestone
8.7

Comments

@alexbevi
Copy link
Contributor

alexbevi commented Jul 5, 2024

Prerequisites

  • I have written a descriptive issue title
  • I have searched existing issues to ensure the feature has not already been requested

🚀 Feature Proposal

Since readyState updates are based on topology status updates, it's less reliable in stateless environments (such as AWS Lamba) as topology update events may be missed while the execution environment is frozen.

This can result in readyState tests returning stale state information.

Motivation

No response

Example

No response
@alexbevi alexbevi added enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature new feature This change adds new functionality, like a new method or class labels Jul 5, 2024
@vkarpov15 vkarpov15 added this to the 8.6 milestone Jul 5, 2024
vkarpov15 added a commit that referenced this issue Aug 17, 2024
@vkarpov15
fix(connection): avoid returning readyState = connected if connecti…
06839fa 
…on state is stale

Fix #14727
@vkarpov15 vkarpov15 mentioned this issue Aug 17, 2024
Merged
@vkarpov15 vkarpov15 modified the milestones: 8.6, 8.7 Aug 26, 2024
@vkarpov15
Copy link
Collaborator

Going to move this one back because I've been having trouble reproducing this issue in AWS lambda

vkarpov15 added a commit that referenced this issue Sep 17, 2024
@vkarpov15
Merge pull request #14812 from Automattic/vkarpov15/gh-14727
89d7b8b 
fix(connection): avoid returning `readyState = connected` if connection state is stale
@nejidevelops nejidevelops mentioned this issue Oct 20, 2024
Open
@ShivamNimje6 ShivamNimje6 mentioned this issue Oct 20, 2024
Open
@zntb zntb mentioned this issue Oct 21, 2024
Merged
@nejidevelops nejidevelops mentioned this issue Oct 22, 2024
Open
@NotZarifGithub NotZarifGithub mentioned this issue Oct 23, 2024
Open
@sharma731 sharma731 mentioned this issue Oct 23, 2024
Open
@axqqt axqqt mentioned this issue Oct 23, 2024
Open
@zntb zntb mentioned this issue Oct 24, 2024
Merged
@Tobsukka Tobsukka mentioned this issue Oct 25, 2024
Open
@ranjith221132 ranjith221132 mentioned this issue Oct 25, 2024
Open
@k2xl k2xl mentioned this issue Oct 25, 2024
Open
@hamzabetbaieb hamzabetbaieb mentioned this issue Oct 25, 2024
Open
@MihikaNigam MihikaNigam mentioned this issue Oct 25, 2024
Open
@AlexanderSCK AlexanderSCK mentioned this issue Oct 25, 2024
Open
@aviraj1703 aviraj1703 mentioned this issue Oct 26, 2024
Open
@yunji0387 yunji0387 mentioned this issue Oct 26, 2024
Merged
@Gabo-Tech Gabo-Tech mentioned this issue Oct 27, 2024
Open
@axqqt axqqt mentioned this issue Oct 27, 2024
Open
@lonewolf0708 lonewolf0708 mentioned this issue Oct 27, 2024
Open
Open
@saileshbro saileshbro mentioned this issue Nov 5, 2024
Merged
@JasperVanNiekerk JasperVanNiekerk mentioned this issue Nov 5, 2024
Merged
@MSK-009 MSK-009 mentioned this issue Nov 5, 2024
Open
@zntb zntb mentioned this issue Nov 6, 2024
Merged
@pradumchintamani pradumchintamani mentioned this issue Nov 6, 2024
Open
@aditya-arcot aditya-arcot mentioned this issue Nov 6, 2024
Merged
@NotZarifGithub NotZarifGithub mentioned this issue Nov 6, 2024
Open
@Erenkrs Erenkrs mentioned this issue Nov 6, 2024
Open
@lholmquist lholmquist mentioned this issue Nov 6, 2024
Merged
@issam-seghir issam-seghir mentioned this issue Nov 6, 2024
Open
@saileshbro saileshbro mentioned this issue Nov 7, 2024
Merged
@Tobsukka Tobsukka mentioned this issue Nov 8, 2024
Open
@k2xl k2xl mentioned this issue Nov 8, 2024
Open
@MihikaNigam MihikaNigam mentioned this issue Nov 8, 2024
Open
@evildevill evildevill mentioned this issue Nov 8, 2024
Closed
@AlexanderSCK AlexanderSCK mentioned this issue Nov 8, 2024
Open
Open
@akanchhaS akanchhaS mentioned this issue Nov 9, 2024
Open
@zntb zntb mentioned this issue Nov 9, 2024
Closed
@aviraj1703 aviraj1703 mentioned this issue Nov 9, 2024
Open
@nejidevelops nejidevelops mentioned this issue Nov 9, 2024
Open
@Gabo-Tech Gabo-Tech mentioned this issue Nov 10, 2024
Open
@axqqt axqqt mentioned this issue Nov 10, 2024
Open
@AndrewTroll AndrewTroll mentioned this issue Nov 11, 2024
Open
Open
@axqqt axqqt mentioned this issue Nov 11, 2024
Open
@sachinthapa572 sachinthapa572 mentioned this issue Nov 11, 2024
Open
@ST10029788 ST10029788 mentioned this issue Nov 12, 2024
Open
@pradumchintamani pradumchintamani mentioned this issue Nov 13, 2024
Open
@hasezoey hasezoey mentioned this issue Dec 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature new feature This change adds new functionality, like a new method or class
Projects
None yet
Milestone
8.7
Development

No branches or pull requests
2 participants
@alexbevi @vkarpov15