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.

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

feat(server): disable UDP service by config #1132

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

GehaFearless
Copy link
Contributor

@GehaFearless GehaFearless commented Aug 22, 2022

issue: #1131

What problem does this PR solve?

Disable UDP service when not use it.

What is changed and how does it work?

Add a configuration 'enable_udp' with a default value of 'false' in the 'network' section. It gives the server control over whether to start the UDP service.

New configuration is added:

[network]
+ enable_udp = true

@github-actions github-actions bot added the cpp label Aug 22, 2022
@GehaFearless GehaFearless changed the title chore(server): disable UDP service by default feat(server): disable UDP service by default Aug 23, 2022
src/server/config.ini Outdated Show resolved Hide resolved
src/server/config.ini Outdated Show resolved Hide resolved
src/rdsn/src/runtime/providers.common.cpp Outdated Show resolved Hide resolved
src/rdsn/src/runtime/nativerun.cpp Outdated Show resolved Hide resolved
@GehaFearless GehaFearless changed the title feat(server): disable UDP service by default feat(server): disable UDP service by config Aug 31, 2022
src/rdsn/src/runtime/nativerun.cpp Outdated Show resolved Hide resolved
src/rdsn/src/runtime/task/task_spec.cpp Outdated Show resolved Hide resolved
@empiredan empiredan merged commit 9a6702a into apache:master Dec 21, 2022
@empiredan empiredan mentioned this pull request Aug 17, 2023
@empiredan empiredan added the type/config-change Added or modified configuration that should be noted on release note of new version. label Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp type/config-change Added or modified configuration that should be noted on release note of new version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants