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.

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

Fix connection method test ios 843 #6875

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

buggmagnet
Copy link
Contributor

@buggmagnet buggmagnet commented Sep 26, 2024

This PR makes it so that only a single instance of EncryptedDNSTransport is ever used in either process.

Important

This should only be merged after #6874


This change is Reviewable

@buggmagnet buggmagnet added the iOS Issues related to iOS label Sep 26, 2024
@buggmagnet buggmagnet self-assigned this Sep 26, 2024
Copy link

linear bot commented Sep 26, 2024

@buggmagnet buggmagnet force-pushed the fix-connection-method-test-ios-843 branch from a9de42c to 1d149f5 Compare September 26, 2024 18:18
Copy link
Contributor

@rablador rablador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Collaborator

@mojganii mojganii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@buggmagnet buggmagnet force-pushed the fix-connection-method-test-ios-843 branch from 1d149f5 to a2c27c2 Compare September 30, 2024 13:21
@buggmagnet buggmagnet merged commit f59440f into main Sep 30, 2024
8 of 9 checks passed
@buggmagnet buggmagnet deleted the fix-connection-method-test-ios-843 branch September 30, 2024 13:27
Copy link

🚨 End to end tests failed. Please check the failed workflow run.

Copy link

🚨 End to end tests failed. Please check the failed workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iOS Issues related to iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants