Skip to content

[STANDARD LIBRARY BREAKING CHANGE] Enhance sys.net: add IP address types, support IPv6, update Host and friends #123

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Frixuu
Copy link

@Frixuu Frixuu commented Nov 12, 2024

The networking-related APIs in the Haxe standard library currently differ significantly
from those found in other languages such as C#, Java, or Rust.
This proposal introduces changes to sys.net to align it with contemporary networking APIs
while addressing several limitations in its current implementation.

<Rendered proposal>

@skial skial mentioned this pull request Nov 14, 2024
1 task
@Frixuu Frixuu changed the title Enhance sys.net: add IP address types, support IPv6, update Host and friends [STANDARD LIBRARY BREAKING CHANGE] Enhance sys.net: add IP address types, support IPv6, update Host and friends Nov 17, 2024
@Frixuu
Copy link
Author

Frixuu commented Nov 27, 2024

Request for voting.

@Aidan63
Copy link
Contributor

Aidan63 commented Jan 4, 2025

If there is a desire to add ipv6 support to the sys socket api then maybe it makes sense to have it utilise parts of the proposed asys api. Asys has support for ipv6 and has a v4 and v6 address enum and various tools very similar to what you've proposed.
Would seem rather silly to have duplicated functionality, maybe a haxe.net.address package for address classes / tools shared between sys and asys?

https://github.com/HaxeFoundation/haxe/blob/900dd50a4c840220cb9218a226d2736123557847/std/asys/native/net/Ip.hx

With asys looking to support ipv6 it might also be easier just to point people at that when they ask about ipv6 support instead of introducing breaking changes to the old sys api.

@dimensionscape
Copy link

dimensionscape commented Feb 19, 2025

With asys looking to support ipv6 it might also be easier just to point people at that when they ask about ipv6 support instead of introducing breaking changes to the old sys api.

To be fair, HaxeFoundation/haxe#6142 issue was closed 5 years ago in favor of the upcoming asys api. But uh.. what is the status of asys these days?

@Frixuu I would be interested in working with someone on an intermediate library to shadow some of the api and fill in the gaps to provide some utility here in the mean time (and provide compatibility with these features for Haxe 4.x.x).

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.

3 participants