Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix the Session with given id not found (closes #7865, #7810) #7875

Merged
merged 3 commits into from
Jul 17, 2023
Merged

fix the Session with given id not found (closes #7865, #7810) #7875

merged 3 commits into from
Jul 17, 2023

Conversation

AlexKamaev
Copy link
Contributor

No description provided.

@AlexKamaev AlexKamaev temporarily deployed to CI July 12, 2023 11:23 — with GitHub Actions Inactive
@AlexKamaev AlexKamaev marked this pull request as ready for review July 13, 2023 07:33
@AlexKamaev AlexKamaev requested a review from miherlosev July 13, 2023 07:33
@AlexKamaev AlexKamaev temporarily deployed to CI July 14, 2023 08:24 — with GitHub Actions Inactive
@AlexKamaev AlexKamaev temporarily deployed to CI July 14, 2023 09:42 — with GitHub Actions Inactive
@Aleksey28 Aleksey28 merged commit aae15b6 into DevExpress:master Jul 17, 2023
@github-actions
Copy link

Release v3.1.0-rc.1 addresses this.

@Artem-Babich Artem-Babich mentioned this pull request Jul 24, 2023
Merged
2 tasks
Aleksey28 pushed a commit that referenced this pull request Jul 24, 2023
@Artem-Babich
fix: Session with given id not found(closes #7865, #7810) (#7897)
acfcf19 
## Purpose
The second part of #7875
We handled the Request error, but we have an unsafe code in
targetAttached handler.

## Approach
Use try-catch 

## References
#7875

## Pre-Merge TODO
- [ ] Write tests for your proposed changes
- [ ] Make sure that existing tests do not fail
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@miherlosev miherlosev miherlosev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AlexKamaev @miherlosev @Aleksey28