-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
Null response and the cookie collection is not working properly #537
Comments
The first error does sound a legitimate bug. Although I'm considering Flurl 2.x to be in maintenance mode, I might consider fixing that one. The second is a known issue with concurrency that I don't plan on fixing. If you're doing concurrent "cookie sessions" in 2.x, my recommendation is to use a Having said all that, I would strongly recommend following #506, a complete revamp of cookies that will be released very soon in the next 3.0 prerelease. It solves these issues and much more, and I really encourage anyone using cookies with Flurl to offer feedback. |
The error I encountered was reported when using Flurl version 3.0 previous 3. |
None of the new cookie bits in in any of the existing prereleases yet. |
#506 is now released in prerelease 4. Please test and let me know if this issue is resolved. |
Well, thank you very much. I will test soon the problem that had previously encountered |
Sure thing. If you confirm it's resolved, are you ok with closing this and #520? I'd just like to clean up my list of open issues a bit by closing anything cookie-related that's a "won't fix" in 2.x and in no longer relevant in 3.0. |
Feel free to re-open if the latest prerelease doesn't resolve this. |
As mentioned in the issue #534 sometimes some requests don't get a response =>
![image](https://user-images.githubusercontent.com/42678520/88453564-5aac2680-ce92-11ea-8851-951a08160d1a.png)
null
From there I realized that a
NullReferenceException
error when reading cookie from responseFlurl.Http.FlurlHttpException: Call failed. Collection was modified; enumeration operation may not execute.
![image](https://user-images.githubusercontent.com/42678520/88453576-6e578d00-ce92-11ea-96d7-4df7893b96d6.png)
The text was updated successfully, but these errors were encountered: