Skip to content

cleanup(vless): remove unused sc-max-concurrent-posts option#2654

Merged
wwqgtxx merged 1 commit into
MetaCubeX:Alphafrom
Nemu-x:cleanup/xhttp-remove-unused-sc-max-concurrent-posts-v2
Mar 29, 2026
Merged

cleanup(vless): remove unused sc-max-concurrent-posts option#2654
wwqgtxx merged 1 commit into
MetaCubeX:Alphafrom
Nemu-x:cleanup/xhttp-remove-unused-sc-max-concurrent-posts-v2

Conversation

@Nemu-x

@Nemu-x Nemu-x commented Mar 29, 2026

Copy link
Copy Markdown

What

Removes the unused sc-max-concurrent-posts field from VLESS XHTTP options.

Why

This option is declared in XHTTPOptions, but unlike the other XHTTP fields, it is not referenced anywhere in the current implementation.

Verified used fields:

  • path
  • host
  • mode
  • headers
  • no-grpc-header
  • x-padding-bytes

Scope

Small cleanup only. No behavior changes intended.

Validation

  • go build ./...
  • go test ./listener/inbound -run XHTTP -v

@wwqgtxx wwqgtxx merged commit 576a442 into MetaCubeX:Alpha Mar 29, 2026
@wwqgtxx

This comment was marked as outdated.

@Nemu-x

This comment was marked as outdated.

@wwqgtxx

wwqgtxx commented Mar 30, 2026

Copy link
Copy Markdown
Collaborator

I realized I made a typo; I was asking about the plan of stream-up mode support.

@Nemu-x

Nemu-x commented Mar 30, 2026

Copy link
Copy Markdown
Author

I realized I made a typo; I was asking about the plan of stream-up mode support.

Thx for clarification

I’m planning to take it as a follow-up and implement it as a separate change.

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