sub SIGNALduino_CheckccConfResponse - make it more robust

[HomeAutoUser]

Expected Behavior

• make it more robust so as not to receive a warning

Actual Behavior

2021.07.08 12:21:08.516 1: PERL WARNING: Illegal hexadecimal digit '%' ignored at ./FHEM/00_SIGNALduino.pm line 1180.

RFFHEM/FHEM/00_SIGNALduino.pm

Lines 1174 to 1181 in a07524a

	sub SIGNALduino_CheckccConfResponse {
	my (undef,$str) = split('=', $_[1]);
	my $var;
	my %r = ( '0D'=>1,'0E'=>1,'0F'=>1,'10'=>1,'11'=>1,'12'=>1,'1B'=>1,'1D'=>1, '15'=>1);
	foreach my $a (sort keys %r) {
	$var = substr($str,(hex($a)-13)*2, 2);
	$r{$a} = hex($var);
	}

Steps to Reproduce the Problem

1. feed the sub with an invalid value (example: 216%E857C43023B900070018146C040091) // value can arise due to an incorrect transmission from serial
2. look at logfile

Specifications

• Microcontroller:
  independent
• Version (Firmware):
  independent

• Versionmodul (FHEM Module):
  3.5.2+20210813

[HomeAutoUser]

Wie denkt Ihr über diesen Fall?
Es gibt ja 2 Möglichkeiten vom Handeln.

1. einfach nichts zurück geben, via return und Ende der Abfrage
   oder
2. die Abfrage wiederholen (birgt aber Gefahr, das man bis hin zu einer Schleife kommen könnte, wenn immer falsche Daten kommen)

[sidey79]

Spontan würde ich zur Verifizierung der Daten tendieren und ein Return zurück geben.

[HomeAutoUser]

Nicht das du denkst @sidey79 wir sind nicht mehr aktiv :-P hier.

Man müsste darüber denken wieweit man das treibt. Ich könnte mir vorstellen nur ein undef bei falsch zurück zu geben und eine Logausgabe für den User. Wenn er mitbekommt, es kommt nichts, schaut dieser ins Log oder probiert es erneut.

Wir stellst du dir eine Verifizierung vor?
Was würde man machen wollen wenn ein nicht korrekter Wert kommt?

[sidey79]

Alles gut, ich bin ja selbst gerade nicht sonderlich aktiv.

Spontan würde mir einfallen dass ein = enthalten sein muss und wir rechts davon nur hex Werte erlauben die in der Anzahl von %r vorhanden sein müssen.

[HomeAutoUser]

@sidey79 ich wollte einen neuen Branch erstellen hier um die Testergebnisse zu sehen. Wurden die Berechtigungen geändert?
Ich würde hierauf HomeAutoUser@98eeb76 aufbauen.

[sidey79]

Ich habe nichts geändert. Du kannst in diesem Repo also keinen neuen Branch erstellen?

[HomeAutoUser]

Nun klappte es. Ich habe es diesmal so https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-and-deleting-branches-within-your-repository gemacht und nur via GitDesktop ging es nicht. Wer weiss ...

[HomeAutoUser]

Dieser Issues wurde erledigt mit #1031