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

Refactored mutex locking in Auth module #1001

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

Pietfried
Copy link
Contributor

@Pietfried Pietfried commented Jan 7, 2025

Describe your changes

  • Passing a std::unique_lock to be able to use it inside a wait_for inside select_evse. Before this change, authorization could've been provided to the same EVSE for parallel authorization requests. With this change, this is not possible anymore
  • Adding a check in the Connector so that an EVSE that already has an identifier assigned is not considered available

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
  • I read the contribution documentation and made sure that my changes meet its requirements

* Passing a std::unique_lock to be able to use it inside a wait_for inside select_evse
* Adding a check in the Connector so that an EVSE that already has an identifier assigned is not considered available

Signed-off-by: Piet Gömpel <[email protected]>
@Pietfried Pietfried force-pushed the refactor/auth-module-lock branch from 54677e5 to 5c8535f Compare January 7, 2025 20:38
@Pietfried Pietfried marked this pull request as ready for review January 7, 2025 20:39
@Pietfried Pietfried added the bug Something isn't working label Jan 8, 2025
@Pietfried Pietfried merged commit c328363 into main Jan 8, 2025
12 checks passed
@Pietfried Pietfried deleted the refactor/auth-module-lock branch January 8, 2025 15:17
Pietfried added a commit that referenced this pull request Jan 15, 2025
* Passing a std::unique_lock to be able to use it inside a wait_for inside select_evse
* Adding a check in the Connector so that an EVSE that already has an identifier assigned is not considered available

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.

2 participants