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

Invalid StartTransaction causes Transactions Page to Error #1609

Open
4 tasks done
brandonprry opened this issue Oct 27, 2024 · 1 comment
Open
4 tasks done

Invalid StartTransaction causes Transactions Page to Error #1609

brandonprry opened this issue Oct 27, 2024 · 1 comment

Comments

@brandonprry
Copy link

brandonprry commented Oct 27, 2024

Checklist

  • I checked other issues already, but found no answer/solution
  • I checked the documentation and wiki, but found no answer/solution
  • I am running the latest version and the issue still occurs
  • I am sure that this issue is about SteVe (and not about the charging station software or something unrelated to SteVe)

Specifications

SteVe Version     : Version:	3.8.0-SNAPSHOT
Operating system  : Docker/OSX
JDK               : Docker/OSX
Database          : MariaDB

Expected Behavior

Page doesn't crash.

Actual Behavior

Page crashes.

image

Steps to Reproduce the Problem

  1. Create a ChargePoint with an ID of 1
  2. Send the following StartTransaction OCPP message
echo '[2, "dddb2599-d678-4ff8-bf38-a230390a1200", "StartTransaction", {"connectorId": 42, "meterStart": 42, "idTag": "some id", "timestamp": "222222017-10-27T19:10:11Z"}]'|websocat -H="Sec-WebSocket-Protocol:ocpp1.6" ws://localhost:8180/steve/websocket/CentralSystemService/1

This is the response I got back, which seems fine.

[3,"dddb2599-d678-4ff8-bf38-a230390a1200",{"transactionId":1,"idTagInfo":{"status":"Invalid"}}]
  1. Check the Transactions page, it should have a stack trace.

Additional context

Thanks.

@juherr
Copy link
Contributor

juherr commented Oct 29, 2024

The issue comes first from the station which sent an invalid message.

Steve could reject the invalid message but it is possible the station will continue the send it.

Maybe Steve should drop the invalid message and notify about the issue.

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

No branches or pull requests

2 participants