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.

Sign up for GitHub

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

Jump to bottom

Improved Error Reporting #140

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Improved Error Reporting #140

merged 1 commit into from
Mar 12, 2024

Conversation

BigThunderSR
Copy link
Owner

@BigThunderSR BigThunderSR commented Mar 12, 2024 •
edited
Loading

  • Logs descriptive errors to console when required properties such as OnStar VIN/username/password/PIN and/or MQTT username/password/polling status topic are not provided at run time.
  • OnStar password/PIN and MQTT password are now masked in the console log output by default.

@BigThunderSR BigThunderSR merged commit 23916bf into main Mar 12, 2024
5 of 7 checks passed
@BigThunderSR BigThunderSR deleted the Improved-Error-Reporting branch March 12, 2024 03:37
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 12, 2024
View reviewed changes
src/index.js Dismissed Show dismissed Hide dismissed
src/index.js Dismissed Show dismissed Hide dismissed
src/index.js Dismissed Show dismissed Hide dismissed
src/index.js Dismissed Show dismissed Hide dismissed
src/index.js Dismissed Show dismissed Hide dismissed
@BigThunderSR BigThunderSR mentioned this pull request Mar 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@BigThunderSR