Skip to content

Conversation

@Pietfried
Copy link
Contributor

@Pietfried Pietfried commented Jun 6, 2025

Describe your changes

Fixed update of OCSP when a new certificate was installed via DataTransfer.req . Timer is used to schedule the update, because handle_data_transfer_pnc_certificate_signed needs to return in order to not block the message queue

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • If OCPP 2.0.1 or OCPP2.1: I have updated the OCPP 2.x status document
  • I read the contribution documentation and made sure that my changes meet its requirements

…nsfer.req . Timer is used to schedule the update, because handle_data_transfer_pnc_certificate_signed needs to return in order to not block the message queue

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried Pietfried requested a review from hikinggrass as a code owner June 6, 2025 13:59
@Pietfried Pietfried added the bug Something isn't working label Jun 6, 2025
@Pietfried Pietfried requested a review from mlitre as a code owner June 6, 2025 13:59
@Pietfried Pietfried merged commit 7c9d31e into main Jun 6, 2025
10 checks passed
mlitre pushed a commit that referenced this pull request Jul 16, 2025
…nsfer.req . Timer is used to schedule the update, because handle_data_transfer_pnc_certificate_signed needs to return in order to not block the message queue (#1097)

Signed-off-by: Piet Gömpel <[email protected]>
@mlitre mlitre deleted the bugfix/ocsp-update-blocks branch July 24, 2025 09:42
Pietfried added a commit that referenced this pull request Nov 19, 2025
…nsfer.req . Timer is used to schedule the update, because handle_data_transfer_pnc_certificate_signed needs to return in order to not block the message queue (#1097)

Signed-off-by: Piet Gömpel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants