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

Sonderzeichen in Attributen werden nicht korrekt codiert #27

Closed
sidamos opened this issue Feb 10, 2019 · 4 comments
Closed

Sonderzeichen in Attributen werden nicht korrekt codiert #27

sidamos opened this issue Feb 10, 2019 · 4 comments

Comments

@sidamos
Copy link

sidamos commented Feb 10, 2019

/addons/xmlapi/protocol.cgi

Die <> im Attribut "values" müssten als < bzw. > codiert werden.

@jens-maus
Copy link
Member

Bitte konkretes Beispiel geben/zeigen und zeigen wie es angeblich "richtiger" sein sollte.

@sidamos
Copy link
Author

sidamos commented Feb 10, 2019

Github hat meinen Text verändert. Mal probieren, ob es jetzt klappt.

Die Servicemeldung ist
<font color=orange>Servicemeldung!</font><br/>- Waschmaschine: Konfigurationsdaten stehen zur Übertragung an.
Das ist legal und funktioniert.

Wenn man jetzt protocol.cgi aufruft, dann erhält man illegales XML:
<row datetime="2019-02-10 11:15:16" names="CCU SV Push Text" values="<font color=orange>Servicemeldung!</font><br/>- Waschmaschine: Konfigurationsdaten stehen zur Übertragung an." timestamp="1549793716" />
error on line 20 at column 70: Unescaped '<' not allowed in attributes values

Korrekt wäre:
<row datetime="2019-02-10 11:15:16" names="CCU SV Push Text" values="&lt;font color=orange&gt;Servicemeldung!&lt;/font&gt;&lt;br/&gt;- Waschmaschine: Konfigurationsdaten stehen zur Übertragung an." timestamp="1549793716" />

Siehe u.a. hier: https://stackoverflow.com/questions/5709232/how-do-i-include-etc-in-xml-attribute-values

@jens-maus
Copy link
Member

@sidamos Da hast du natürlich vollkommen recht. Allerdings passiert das nur weil du den entsprechenden Skript hast der deine "CCU SV Push Text" variable mit html Codes füllt. Standardmäßig passiert sowas nicht auf einer Vanille CCU bzw. RaspberryMatic. Es ist aber richtig das hier die Funktion entsprechend die html tags ersetzen sollte

@sidamos
Copy link
Author

sidamos commented Feb 10, 2019

Beides korrekt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants