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

Datenpunkte mit Sonderzeichen in vis in bindings #807

Closed
wants to merge 1 commit into from
Closed

Datenpunkte mit Sonderzeichen in vis in bindings #807

wants to merge 1 commit into from

Conversation

oweitman
Copy link
Contributor

Now the same change for vis2
like #802

  1. und 3. Datenpunkte innerhalb eines bindings, 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

Now the same change for vis2
like #802

2. und 3. Datenpunkte innerhalb eines bindings, 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]>
}

// 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 '-'.
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
' can be anywhere in the URL, and arbitrary hosts may come before or after it.
@oweitman
Copy link
Contributor Author

i dont understand the result of the PR
why all lines are recognized as add?
i only changed line 125

@GermanBluefox
Copy link
Contributor

Becasue it is file from vis-2

@oweitman
Copy link
Contributor Author

oweitman commented Nov 29, 2023

yes,
and i edit this file in the vis-2 repository.
but then the PR points to the vis1 repo
very strange

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