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

feat: add status codes #18

Merged
merged 3 commits into from
Aug 6, 2024
Merged

Conversation

muneebkq
Copy link
Collaborator

No description provided.

@muneebkq muneebkq requested a review from ahmad-kemsan July 31, 2024 12:06
@ahmad-kemsan
Copy link
Collaborator

@muneebkq why are there so many changes to this file? Why was '=' replaced with ':'?

Copy link
Collaborator

@ahmad-kemsan ahmad-kemsan left a comment

Choose a reason for hiding this comment

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

Please rectify the changes

@ahmad-kemsan
Copy link
Collaborator

@muneebkq You seem to have skipped the matching of error code with the lexfloatclient exceptions


LF_E_WMIC = 65

LF_E_MACHINE_FINGERPRINT = 66

Copy link
Collaborator

Choose a reason for hiding this comment

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

@muneebkq Please add 67 as well

if code == LexFloatStatusCodes.LF_E_WMIC:
return "Fingerprint couldn't be generated because Windows Management Instrumentation (WMI) service has been disabled. This error is specific to Windows only."
if code == LexFloatStatusCodes.LF_E_MACHINE_FINGERPRINT:
return 'Machine fingerprint has changed since activation.'
return 'Unknown error!'
Copy link
Collaborator

Choose a reason for hiding this comment

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

@muneebkq Please add the invalid proxy error code as well

Copy link
Collaborator

@ahmad-kemsan ahmad-kemsan left a comment

Choose a reason for hiding this comment

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

Please add 67 as well

@muneebkq muneebkq requested a review from ahmad-kemsan August 6, 2024 06:24
@ahmad-kemsan ahmad-kemsan merged commit 8f456ca into borrowing-develop Aug 6, 2024
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