Skip to content

bluetooth: Normalize HCI connection_handle field names to be accordin… - #5176

Merged
polybassa merged 1 commit into
secdev:masterfrom
antoniovazquezblanco:normhandle
Sep 12, 2026
Merged

polybassa merged 1 commit into
secdev:masterfrom
antoniovazquezblanco:normhandle

Conversation

@antoniovazquezblanco

Copy link
Copy Markdown
Contributor

…g spec.

Add deprecation warnings for backwards compat.

AI-Assisted: yes (Claude Opus 5)

Description

…g spec.

Add deprecation warnings for backwards compat.

AI-Assisted: yes (Claude Opus 5)
@antoniovazquezblanco

Copy link
Copy Markdown
Contributor Author

@XenoKovah, I would be interested in your input and if you agree with this change. Thanks!

@codecov

codecov Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.14%. Comparing base (251daec) to head (7ce1549).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5176      +/-   ##
==========================================
+ Coverage   81.13%   81.14%   +0.01%     
==========================================
  Files         392      392              
  Lines       97843    97865      +22     
==========================================
+ Hits        79384    79413      +29     
+ Misses      18459    18452       -7     
Files with missing lines Coverage Δ
scapy/layers/bluetooth.py 91.12% <100.00%> (+0.15%) ⬆️

... and 6 files with indirect coverage changes

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

@XenoKovah

Copy link
Copy Markdown
Contributor

This is less important to me than the bd_addr change, but since it does map to the exact names used in the spec I think it's worth doing.

@polybassa
polybassa merged commit bc6a366 into secdev:master Sep 12, 2026
23 checks passed
@antoniovazquezblanco
antoniovazquezblanco deleted the normhandle branch September 12, 2026 14:25
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.

3 participants