Skip to content

Conversation

@Jolymmiles
Copy link

No description provided.

@wwqgtxx wwqgtxx changed the base branch from Meta to Alpha November 24, 2025 13:52
Copy link
Collaborator

@wwqgtxx wwqgtxx left a comment

Choose a reason for hiding this comment

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

As we mentioned in #2383, if the PR could include a server-side implementation (i.e., the code for listener and listener\inbound_test), it might prompt us to merge the code as soon as possible.

tlsConfig.NextProtos = []string{"h2", "http/1.1"}
}

// Handle ECH and Reality if needed (skipped for now or requires more adaptation)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually, you can pass mihomoVMess.TLSConfig just like the transport layers in h2, which would simplify the handling of reality and ech.

}

var (
globalDialerMap map[dialerConf]*XmuxManager
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we really need a package-level globalDialerMap instead of a *Client object that directly stores these values ​​when creating *Vless and *Vmess?

Copy link

@Fangliding Fangliding Nov 28, 2025

Choose a reason for hiding this comment

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

Do we really need a package-level globalDialerMap instead of a *Client object that directly stores these values ​​when creating *Vless and *Vmess?

obviously (ai) copied from *ray because they have no struct for a transport to hold cross-session information(


c, err = vmess.StreamH2Conn(ctx, c, h2Opts)
case "splithttp", "xhttp":
c.Close()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why not implement this in DialContext or DialContextWithDialer (like the gRPC transport layer)? Creating a new conn and then immediately closing it would result in confusing traffic patterns.

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