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

Extended Use of SecurityEventNotification.req in OCPP1.6 #425

Merged
merged 4 commits into from
Jan 31, 2024

Conversation

Pietfried
Copy link
Contributor

  • Sending SecurityEventNotification in OCPP1.6 on startup if StartUpOfTheDevice or ResetOrReboot
  • added optional BootReason to charge_point start function
  • added optional BootReason to charge_point restart function
  • using string definitions for all SecurityEventNotifications in 1.6

* added optional BootReason to charge_point restart function
* using string definitions for all SecurityEventNotifications in 1.6
* Sending SecurityEventNotification on startup if StartUpOfTheDevice or ResetOrReboot

Signed-off-by: pietfried <pietgoempel@gmail.com>
@Pietfried Pietfried force-pushed the feature/extend-ocpp16-security-events branch from ffa6d06 to a52207a Compare January 30, 2024 10:17
Signed-off-by: pietfried <pietgoempel@gmail.com>
@Pietfried
Copy link
Contributor Author

Pietfried commented Jan 30, 2024

Before merge:

  • Introduce configuration key for 1.6 to control if SecurityEventNotification.req should be send or not

…an be used to disable the sending of SecurityEventNotification.req

Signed-off-by: pietfried <pietgoempel@gmail.com>
…tifications configuration key is false (which is the default)

Signed-off-by: pietfried <pietgoempel@gmail.com>
@Pietfried Pietfried requested a review from hikinggrass January 30, 2024 21:12
@corneliusclaussen corneliusclaussen merged commit fde2ece into main Jan 31, 2024
4 checks passed
@corneliusclaussen corneliusclaussen deleted the feature/extend-ocpp16-security-events branch January 31, 2024 08:53
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.

3 participants