Skip to content

feat: install and activate the tunnel provider as network extension #20

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

Merged
merged 3 commits into from
Jan 14, 2025

Conversation

spikecurtis
Copy link
Collaborator

@spikecurtis spikecurtis commented Jan 10, 2025

With apologies, I'm sort of punting on this one for the time being.

The code works to install and allow activation of the Network Extension, but for some reason the delegate I've created to get updates about the System Extension installation is not getting called back.

I've posted on the Apple Developer Forums about it: https://forums.developer.apple.com/forums/thread/772236

In the meantime, I'm hoping to get some feedback about the code and overall design. I'll leave it in draft for now.

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@spikecurtis spikecurtis force-pushed the spike/network-extension-install branch 2 times, most recently from 3b95b81 to 1683ca6 Compare January 10, 2025 11:20
Copy link
Member

@ethanndickson ethanndickson left a comment

Choose a reason for hiding this comment

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

lgtm so far!

@ethanndickson
Copy link
Member

Rebasing...

@ethanndickson ethanndickson force-pushed the spike/network-extension-install branch from 1683ca6 to 9e0b05b Compare January 14, 2025 05:30
@ethanndickson
Copy link
Member

Merging so Colin can just work off main

@ethanndickson ethanndickson marked this pull request as ready for review January 14, 2025 05:43
@ethanndickson ethanndickson merged commit 90d124c into main Jan 14, 2025
4 checks passed
@ethanndickson ethanndickson deleted the spike/network-extension-install branch January 23, 2025 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants