Skip to content

Doip server initial code #4

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

Open
VinaykumarRS1995 wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Doip server initial code #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
04327e2
feat(doip): implement header parsing with tokio codec
VinaykumarRS1995 Jan 30, 2026
a9bd564
test(doip): add unit tests for header_parser
VinaykumarRS1995 Feb 2, 2026
49e0f7e
feat(error): add DoIP error types and NRC codes
VinaykumarRS1995 Feb 2, 2026
2208cbb
feat(uds): add UdsHandler trait for pluggable backends
VinaykumarRS1995 Feb 2, 2026
1d5e99a
refactor(doip): address PR review comments and Improvement
VinaykumarRS1995 Feb 25, 2026
b12490a
style: apply cargo fmt
VinaykumarRS1995 Mar 3, 2026
7aefa63
refactor(doip): address PR review comments
VinaykumarRS1995 Mar 5, 2026
6c0d47f
refactor: fix PR review
VinaykumarRS1995 Mar 13, 2026
ceb170d
chore: move doip-server into Cargo workspace to resolve merge conflict
VinaykumarRS1995 Mar 17, 2026
b98ff03
fix(review): address review comments
VinaykumarRS1995 Mar 20, 2026
dfe3c85
fix(review): address .toml config, empty data validation.
VinaykumarRS1995 Mar 30, 2026
6d8eab2
fix(review): add unit tests, trace logging
VinaykumarRS1995 Mar 30, 2026
eff3242
fix(review): revert CI SHAs, use constants in vehicle_id, fix doc and…
VinaykumarRS1995 Apr 21, 2026
1eb7c2b
fix(review): address review comments
VinaykumarRS1995 Apr 21, 2026
5cd28b3
fix(review): address second round of review comments
VinaykumarRS1995 May 11, 2026
e1039b4
docs: rename diagrams, refactor sequence diagram, add rendered SVGs
VinaykumarRS1995 May 12, 2026
53214b3
fix(review): address reviewer comments on doip-server-initial
VinaykumarRS1995 May 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
297 changes: 296 additions & 1 deletion Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

36 changes: 31 additions & 5 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,31 @@
# terms of the Apache License Version 2.0 which is available at
# https://www.apache.org/licenses/LICENSE-2.0

[package]
name = "uds2sovd-proxy"
version = "0.1.0"
[workspace.package]
edition = "2024"
license = "Apache-2.0"
homepage = "https://github.com/eclipse-opensovd/uds2sovd-proxy"

[lints]
workspace = true
[workspace]
resolver = "3"
members = ["doip-server"]

[workspace.dependencies]
# ---- async runtime ----
tokio = { version = "1", default-features = false }
tokio-util = { version = "0.7", default-features = false }
# ---- serialization ----
serde = { version = "1.0", default-features = false }
toml = { version = "0.9", default-features = false }
# ---- tracing & logging ----
tracing = { version = "0.1", default-features = false }
# ---- error handling ----
thiserror = { version = "2.0", default-features = false }
# ---- data structures ----
bytes = { version = "1.0", default-features = false }
parking_lot = { version = "0.12", default-features = false }
# ---- utility crates ----
hex = { version = "0.4", default-features = false }

# Lints are aligned with shared-lints.toml from:
# https://github.com/eclipse-opensovd/cicd-workflows/tree/main/shared-lints

@arvindr19 arvindr19 Mar 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is the workspace Cargo.toml, so this information shouldn’t be removed. I’d suggest keeping these changes in the package-level Cargo.toml instead. I also noticed that the comments below were removed, please keep them intact.

@VinaykumarRS1995 VinaykumarRS1995 Mar 30, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Restored — lint header comment and all per-lint explanatory comments are kept intact.

Expand Down Expand Up @@ -44,3 +60,13 @@ clone_on_ref_ptr = "warn"
# enforce that the type suffix of a literal is always appended directly
# eg. 12u8 instead of 12_u8
separated_literal_suffix = "deny"

[workspace.lints.rust]
unexpected_cfgs = { level = "warn", check-cfg = ['cfg(nightly)'] }

[profile.release.package."*"]
opt-level = 3

[profile.release-with-debug]
inherits = "release"
debug = true
9 changes: 8 additions & 1 deletion deny.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,14 @@ ignore = []
allow = ["Apache-2.0", "MIT"]

confidence-threshold = 0.9
exceptions = []
exceptions = [
# unicode-ident is a transitive dependency pulled in by serde (via proc-macro2).
# It is licensed under Unicode-3.0, an OSI-approved license covering only
# Unicode data tables. There is no Apache-2.0 or MIT alternative for this crate.
{ allow = [
"Unicode-3.0",
], name = "unicode-ident" },
]

[licenses.private]
ignore = true
Expand Down
Loading