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.

Sign up for GitHub

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

Jump to bottom

Override inbounds in multi config files #2655

Merged
merged 3 commits into from Oct 23, 2023
Merged

Override inbounds in multi config files #2655

merged 3 commits into from Oct 23, 2023

Conversation

hossinasaadi
Copy link
Contributor

when use multi config file inbounds not appends :
./xray -c config.json -c config2.json -c config3.json
issue mentioned in #2606

@yuhan6665
Copy link
Member

thanks for your pr! please take a look at this test TestConfig_Override

@hossinasaadi
Copy link
Contributor Author

thanks for your pr! please take a look at this test TestConfig_Override

@yuhan6665 seems test fails cause of wrong test.
i think with this PR, Inbound Overriding work much better, right?

@yuhan6665 yuhan6665 merged commit 71fee07 into XTLS:main Oct 23, 2023
34 of 35 checks passed
@yuhan6665
Copy link
Member

Agree. Let's give it a try :)

@hossinasaadi hossinasaadi deleted the overrideInbounds branch October 23, 2023 08:14
@hossinasaadi hossinasaadi mentioned this pull request Oct 23, 2023
Merged
@BioniCosmos BioniCosmos mentioned this pull request Feb 13, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@hossinasaadi @yuhan6665