Skip to content

Ignore attributes after MESSAGE-INTEGRITY in a STUN message.#309

Merged
JonathanLennox merged 1 commit intojitsi:masterfrom
JonathanLennox:ignore-attributes-after-mi
Apr 22, 2026
Merged

Ignore attributes after MESSAGE-INTEGRITY in a STUN message.#309
JonathanLennox merged 1 commit intojitsi:masterfrom
JonathanLennox:ignore-attributes-after-mi

Conversation

@JonathanLennox
Copy link
Copy Markdown
Member

(Other than FINGERPRINT).

Add unit tests for message integrity processing, including this behavior.

(Other than FINGERPRINT).

Add unit tests for message integrity processing, including this behavior.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.30%. Comparing base (8e0db06) to head (60afa6a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #309      +/-   ##
============================================
+ Coverage     35.84%   36.30%   +0.46%     
- Complexity     1125     1145      +20     
============================================
  Files           166      166              
  Lines         10993    11037      +44     
  Branches       1627     1629       +2     
============================================
+ Hits           3940     4007      +67     
+ Misses         6524     6508      -16     
+ Partials        529      522       -7     
Files with missing lines Coverage Δ
src/main/java/org/ice4j/message/Message.java 50.90% <100.00%> (+0.72%) ⬆️

... and 13 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e0db06...60afa6a. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JonathanLennox JonathanLennox merged commit 6da2b08 into jitsi:master Apr 22, 2026
6 checks passed
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