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

Datenpunkte mit Sonderzeichen in vis in bindings #803

Closed
wants to merge 1 commit into from
Closed

Datenpunkte mit Sonderzeichen in vis in bindings #803

wants to merge 1 commit into from

Conversation

oweitman
Copy link
Contributor

  1. und 3. Datenpunkte innerhalb eines Datenpunktes, welche Sonderzeichen enthalten (bspw öäü oder #) konnten in vis nicht für Berechnungen in bindings verwendet werden, da sie durch das regex nicht erkannt werden konnten. das regex wurde aus dem js-controller entnommen, der dort auf gültige datenpunktbezeichnungen prüft. https://forum.iobroker.net/topic/69004/rechnen-im-html-widget-geht-nicht-mehr/19?page=1

@oweitman
Datenpunkte mit Sonderzeichen in vis in bindings
8f6a280 
2. und 3. Datenpunkte innerhalb eines Datenpunktes, welche Sonderzeichen enthalten (bspw öäü oder #) konnten in vis nicht für Berechnungen in bindings verwendet werden, da sie durch das regex nicht erkannt werden konnten. das regex wurde aus dem js-controller entnommen, der dort auf gültige datenpunktbezeichnungen prüft.
https://forum.iobroker.net/topic/69004/rechnen-im-html-widget-geht-nicht-mehr/19?page=1

Signed-off-by: oweitman <[email protected]>
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 19, 2023
View reviewed changes
src/src/Vis/visUtils.jsx
}

// new style: %html%, %myAttr%, ...
ms = inputStr.match(/%([-_a-zA-Z\d]+)+?%/g);

Check failure

Code scanning / CodeQL

Inefficient regular expression High

This part of the regular expression may cause exponential backtracking on strings starting with '%' and containing many repetitions of '-'.
src/src/Vis/visUtils.jsx
script.setAttribute('data-webpack', `${remote}`);
script.async = true;
script.onerror = () => {
if (!remoteFallbackUrl.includes('iobroker.net')) {

Check failure

Code scanning / CodeQL

Incomplete URL substring sanitization High

'
iobroker.net
Loading
' can be anywhere in the URL, and arbitrary hosts may come before or after it.
@oweitman oweitman closed this Nov 19, 2023
@oweitman oweitman deleted the patch-2 branch November 19, 2023 14:43
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
@oweitman