Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
This repository was archived by the owner on Dec 5, 2021. It is now read-only.
/TLSharpPublic archive

add handler for exception with code 32#910

Open
n3nikita wants to merge1 commit intosochix:master
base:master
Choose a base branch
Loading
fromn3nikita:issue_668

Conversation

n3nikita
Copy link

Add handler for exception with code 32.#668 and#777 issues.

@knocte
Copy link
Collaborator

What does this achieve? It just says "add handler", does this mean it is the fix for#668 and#777?

@n3nikita
Copy link
Author

Yes. I've just PR the code from issue#668 which should fix error with code 32.

@knocte
Copy link
Collaborator

This is very weird code.

@CheshireCaat can you review?

@CheshireCaat
Copy link
Contributor

I think there is a more correct way to get rid of this error. How can I repeat it to make sure that this solution is right?

@CheshireCaat
Copy link
Contributor

Okay, I looked at issues related to this problem, I think we need a more global solution for working with parallel requests as well.
Maybe it will be better to add something like SemaphoreSlim to theSendRequestAsync orRequestWithDcMigration?

@CheshireCaat
Copy link
Contributor

Now the code from the example below will throw errors

awaitTask.WhenAll(Enumerable.Range(1,5).Select(arg=>Client.CheckUsernameAsync("CheshireCaat")))

I tested it also with modifiedRequestWithDcMigration withSemaphoreSlim - it works fine.
Can you check your error by adding Semaphore? If this helps, I think it's better to do it that way.

@CheshireCaat
Copy link
Contributor

@n3nikita ping

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers
No reviews
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
@n3nikita@knocte@CheshireCaat

[8]ページ先頭

©2009-2025 Movatter.jp