Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update VIN regex #593

Merged
merged 1 commit into from
Jan 23, 2024
Merged

Conversation

rikroe
Copy link
Member

@rikroe rikroe commented Jan 22, 2024

Proposed change

Fixes VIN pseudonomization for vehicles not built in Germany (starting with WB...).

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality to this library)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • The code change is tested and works locally.
  • Tests have been added to verify that the new code works.

Copy link

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9d3bd67) 100.00% compared to head (dcea1a9) 100.00%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #593   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           17        17           
  Lines         1412      1412           
=========================================
  Hits          1412      1412           
Flag Coverage Δ
3.10 100.00% <100.00%> (ø)
3.11 100.00% <100.00%> (ø)
3.12 100.00% <100.00%> (ø)
3.8 100.00% <100.00%> (ø)
3.9 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rikroe rikroe requested a review from gerard33 January 22, 2024 19:55
@rikroe rikroe merged commit 8498511 into bimmerconnected:master Jan 23, 2024
13 checks passed
@rikroe rikroe deleted the fix-vin-anonymization branch December 25, 2024 12:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BMW Connected Drive leaks VIN in diagnostics
2 participants