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

Crashing when session send response with post close #107

Closed
hrnozel opened this issue Nov 8, 2021 · 7 comments
Closed

Crashing when session send response with post close #107

hrnozel opened this issue Nov 8, 2021 · 7 comments

Comments

@hrnozel
Copy link

hrnozel commented Nov 8, 2021

First of all, thank you for your efforts.

I think your library is great and working as properly in Win10 and as I have tested it, It never crashed to now.

But My application is a cross-platform application and it needs to be run on Macintosh and Linux distros.

I have tried my code when I did not any changes in my codes, your library was crashed. I investigated to find what was going to be wrong. I found that if I send response with "post-close" your library will be crashed.

If you want me to get any information such as core dump, debug info I can send it.

My compiler is AppleClang 13.0.0.13000029.
My code is here that how to send my response:

session->send(httpResponse.getResult(), [session]() { session->postClose(); });

@IronsDu
Copy link
Owner

IronsDu commented Nov 8, 2021

@thirtyninekilovolts thans for you found this issue.
I fix it on this commit: 9992477.
please try again. Thank you.

@hrnozel
Copy link
Author

hrnozel commented Nov 9, 2021

@IronsDu Thank you for your interest, it seems that the issue is solved.

@hrnozel hrnozel closed this as completed Nov 9, 2021
@hrnozel
Copy link
Author

hrnozel commented Nov 12, 2021

This issue exists in Linux distros. But it resolved in that commit 9992477, just a piece of information.

@IronsDu
Copy link
Owner

IronsDu commented Nov 13, 2021

@thirtyninekilovolts you said is : publish a release version?

@IronsDu
Copy link
Owner

IronsDu commented Nov 13, 2021

This issue exists in Linux distros. But it resolved in that commit 9992477, just a piece of information.

The master branch still exist this issue?

@hrnozel
Copy link
Author

hrnozel commented Nov 13, 2021

This issue exists in Linux distros. But it resolved in that commit 9992477, just a piece of information.

The master branch still exist this issue?

I meant that was just to say that this bug is also present in Linux distributions so there is no longer exists a bug related to my issue. It was resolved in your commit.

@IronsDu
Copy link
Owner

IronsDu commented Nov 13, 2021

@thirtyninekilovolts
Oh, now, I understand.. I n fact, I fix the issue under my WSL subsystem. hahahah, thank you again.

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

No branches or pull requests

2 participants