-
Notifications
You must be signed in to change notification settings - Fork 4k
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
API - Add | Remove Routing Rules #3189
Conversation
It seems the test has failed |
How about replace the entire routing object? That is easier to implement and use. More over we can hot-switch some balancing strategies too. |
now should be ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR! This is an important new feature. I've made some comments. Looking forward to it
apply balancer config
hot-switch balancer and replace|append new routing configs added. |
Thanks a lot! When I have time, I will integrate #412 as the next step. (Primarily to support GetRoutingRules API) |
glad to hear that.also i will check and see how can i help with that. |
* main: (24 commits) Add "nosni" option to send empty SNI (XTLS#3214) API: add Source IP Block command (XTLS#3211) v1.8.10 Fix TestXrayConfig in xray_test.go Add separate host config for websocket Update proto file for websocket and httpupgrade (breaking) API - Add | Remove Routing Rules (XTLS#3189) Fix host in headers field does not work XTLS#3191 fix: config `burstObservatory` override Bump github.com/sagernet/sing from 0.3.6 to 0.3.8 Add support for HTTPupgrade custom headers improve balancer_info.go Fix(httpupgrade): `X-Forwarded-For` header not read. (XTLS#3172) Allow to send through random IPv6 Update HTTPUpgrade spelling and proto Chore: Clean up legacy `field` usage Update README.md Bump github.com/quic-go/quic-go from 0.41.0 to 0.42.0 Fix HTTPUpgrade transport register HTTPUpgrade 0-RTT (XTLS#3152) ...
I run the command line and get this error |
"api": {
"services": [
"RoutingService"
]
},
|
Thanks |
This reverts commit 53e5814. Because this commit conflict with api getr/setr.
add rules :
./xray api adrules --server=127.0.0.1:62789 c1.json c2.json
remove rules :
./xray api rmrules --server=127.0.0.1:62789 ruleTag1 ruleTag2
also add ruleTag to identify rules.