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

Use key when checking if session is in the Map. #209

Merged
merged 1 commit into from
Sep 19, 2022

Conversation

dimaa6
Copy link

@dimaa6 dimaa6 commented Sep 15, 2022

Fixes #208. We already have session ID in session object, and can use it to make sure session is still in the Map keys. There is no need to check if the session object itself is in the Map values.

@TVolden TVolden merged commit 44def2e into ChargeTimeEU:master Sep 19, 2022
@TVolden
Copy link
Member

TVolden commented Sep 19, 2022

Sorry for my slow response.
I have now merged the change.
Thanks for your contribution, it really means a lot to me.

Sincerely,
Thomas

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.

To obtain sessionId, sessions map is sought for value
3 participants