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

Weitergeleitete Telegramme werden nicht geacked #8

Closed
Ing-Dom opened this issue Feb 11, 2024 · 1 comment
Closed

Weitergeleitete Telegramme werden nicht geacked #8

Ing-Dom opened this issue Feb 11, 2024 · 1 comment
Assignees
Labels
bug Something isn't working fixed_in_head this bug is already fixed in the repo head

Comments

@Ing-Dom
Copy link
Member

Ing-Dom commented Feb 11, 2024

Waldemar (mumpf)
01:12 Uhr
Hmm, was der joeknx123 zum Router geschrieben hat, hört sich ganz nach fehlendem ACK an. Er hat alles auf Durchzug, damit müsste jedes weitergeleitete Telegramm bestätigt werden. Damit sollte es keine Wiederholungen auf TP-Seite geben.
Ich weiß nicht, wie gut Dein Algorithmus für "nur weitergeleitete Telegramme bestätigen" ist, aber Du könntest bei "alle weiterleiten" auf jeden Fall auch intern als "alle Bestätigen" interpretieren.

Ing-Dom (Dominik)
01:15 Uhr
https://github.com/OpenKNX/knx/blob/rp2040_ip_routing_debugCemiFrame/src/knx/bau091A.cpp#L161-L217

bau091A.cpp
TPAckType Bau091A::isAckRequired(uint16_t address, bool isGrpAddr)
{
//only called from TpUartDataLinkLayer
TPAckType ack = TPAckType::AckReqNone;

Mehr anzeigen
https://github.com/[OpenKNX/knx](https://github.com/OpenKNX/knx)|OpenKNX/knxOpenKNX/knx | Hinzugefügt von GitHub

01:17 Uhr
hmmmm
01:18 Uhr
das könnte sein

Waldemar (mumpf)
01:19 Uhr
Da wird doch nur in die Filtertabelle geschaut...

Ing-Dom (Dominik)
01:20 Uhr
wenn "nur weitergeleitete Telegramme bestätigen" ausgewählt ist, prüft er die Filtertabelle. Da ist es aber ggf nicht drin, wenn die gruppentelegramme gar nicht gefiltert werden
01:21 Uhr
die Fallunterscheidung hatte ich nicht am Schirm. Die muss da wohl noch rein

Waldemar (mumpf)
01:21 Uhr
Aber bei "alles Weiterleiten" reicht da nicht, oder? Ich bin mir sicher, dass er nicht "alle Gruppentelegramme bestätigen" angekreuzt hat. Das könnte man ihm als Workaround vorschlagen.

Ing-Dom (Dominik)
01:22 Uhr
so wie es aktuell implementiert ist leitet er die telegramme zwar weiter, acked sie aber nicht
01:22 Uhr
das entspricht nicht der Erwartung und ist auch wenig sinnvoll..

@Ing-Dom Ing-Dom added the bug Something isn't working label Feb 11, 2024
@Ing-Dom Ing-Dom self-assigned this Feb 11, 2024
Ing-Dom added a commit to OpenKNX/knx that referenced this issue Feb 11, 2024
when a frame must be acked or not, not only the routing table is evaluated but also if the router is configured to forward
@Ing-Dom Ing-Dom added the fixed_in_head this bug is already fixed in the repo head label Feb 11, 2024
@Ing-Dom
Copy link
Member Author

Ing-Dom commented Feb 14, 2024

fixed with 0.1.2

@Ing-Dom Ing-Dom closed this as completed Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed_in_head this bug is already fixed in the repo head
Projects
None yet
Development

No branches or pull requests

1 participant