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

Add decrypt value guard #1861

Closed
Apollon77 opened this issue Mar 20, 2022 · 2 comments · Fixed by #1917
Closed

Add decrypt value guard #1861

Apollon77 opened this issue Mar 20, 2022 · 2 comments · Fixed by #1917
Labels
adapter related 🔩 code enhancement 👨‍💻
Projects
Next version 4.1 (...)

Comments

@Apollon77
Copy link
Collaborator

https://sentry.iobroker.net/share/issue/f38e6e83f3bb431eb9039e8e8f529ed6/

in fact value in tools.decrypt must be checked that it is string before using startsWith :-)
@Apollon77 Apollon77 added this to Planned in Next version 4.1 (...) via automation Mar 20, 2022
@Apollon77 Apollon77 changed the title Add decrypt guard Add decrypt value guard Mar 20, 2022
@iobroker-sentry-github-app
Copy link

Sentry issue: IOBROKER-EMAIL-S

@foxriver76
Copy link
Collaborator

Will be fixed with ts port of adapter

@foxriver76 foxriver76 mentioned this issue Jun 6, 2022
Merged
Next version 4.1 (...) automation moved this from Planned to Done Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
adapter related 🔩 code enhancement 👨‍💻
Projects
No open projects
Next version 4.1 (...)
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

started adapter.ts port ioBroker/ioBroker.js-controller
2 participants
@Apollon77 @foxriver76