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

BridgedNetwork implementation #151

Merged
merged 5 commits into from
Dec 20, 2023
Merged

BridgedNetwork implementation #151

merged 5 commits into from
Dec 20, 2023

Conversation

Raikerian
Copy link
Contributor

Which issue(s) this PR fixes:

Found one issue related to that #138

Additional documentation

Added implementation for BridgedNetwork in order to be able to create network device using bridge network, rather than NAT.
Using BridgedNetwork does indeed require special entitlements from Apple, which is a whole PIA to setup and test by itself, as you need proper certificates and signing in place after you got it added to your Organization by Apple. My current project does have those entitlements and needs bridged network, so I was able to test this and it does the job. Please let me know your thoughts if anything needs to be improved.

virtualization_11.m Outdated Show resolved Hide resolved
virtualization_11.m Outdated Show resolved Hide resolved
internal/objc/objc.go Outdated Show resolved Hide resolved
network.go Outdated Show resolved Hide resolved
@Raikerian
Copy link
Contributor Author

@Code-Hex apologies for the delay, all the comments are now addressed

Copy link
Owner

@Code-Hex Code-Hex left a comment

Choose a reason for hiding this comment

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

Thanks

@Code-Hex Code-Hex merged commit 4f8ae6d into Code-Hex:main Dec 20, 2023
4 of 7 checks passed
@Code-Hex
Copy link
Owner

I will release tomorrow in JST

@Raikerian
Copy link
Contributor Author

hey @Code-Hex , version bump still planned?

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