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

fix: update device UUID extraction logic in BaseService #98

Merged
merged 4 commits into from
Dec 8, 2024

Conversation

SecKatie
Copy link
Owner

@SecKatie SecKatie commented Dec 8, 2024

Changed the method of extracting the device UUID from the MAC address in the BaseService class. The extraction now uses the last segment of the MAC address instead of the third segment, ensuring correct UUID retrieval for device control and info requests.

Changed the method of extracting the device UUID from the MAC address in the BaseService class. The extraction now uses the last segment of the MAC address instead of the third segment, ensuring correct UUID retrieval for device control and info requests.
@SecKatie SecKatie self-assigned this Dec 8, 2024
Copy link
Owner Author

@SecKatie SecKatie left a comment

Choose a reason for hiding this comment

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

Fixes #71

@SecKatie SecKatie merged commit b8e4d39 into main Dec 8, 2024
4 checks passed
@SecKatie SecKatie deleted the SecKatie/issue71 branch December 8, 2024 03:45
@JoeSchubert
Copy link
Collaborator

Sorry I just saw the emails about this. I don't see anything in the code that should be an issue. As long as the last segment will be the right segment for everything. (Not sure as I don't use wyze devices much lately)

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