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
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Oauth session#242

Open
avdb13 wants to merge44 commits intosugyan:main
base:main
Choose a base branch
Loading
fromavdb13:oauth-session
Open

Oauth session#242

avdb13 wants to merge44 commits intosugyan:mainfromavdb13:oauth-session

Conversation

avdb13
Copy link
Contributor

No description provided.

@sugyan
Copy link
Owner

Thank you for the pull-request!
This is very helpful that you complemented the parts that I had not yet implemented, such as token revoke and refresh, etc..

TheCached part is a faithful port of theCachedGetter from the TypeScript SDK to Rust...? I haven't tried it yet, but it would be a difficult code...
I imagine that we want to prevent the same request from being executed multiple times in parallel like theThrottledResolver I introduced withatrium-identity. I think it would be nice if we could somehow bundle them in a common form....

FlorianDevPhynix and avdb13 reacted with thumbs up emojiavdb13 reacted with heart emoji

@avdb13avdb13force-pushed theoauth-session branch 6 times, most recently fromb5f94d0 to1f9f9d8CompareNovember 12, 2024 06:33
@avdb13avdb13 marked this pull request as ready for reviewNovember 12, 2024 06:53
@avdb13avdb13force-pushed theoauth-session branch 2 times, most recently from078cb0d to465c57eCompareNovember 14, 2024 22:11
@avdb13avdb13 changed the base branch frommain tofeature/agent-reworkNovember 14, 2024 22:13
@avdb13avdb13 changed the base branch fromfeature/agent-rework tomainNovember 14, 2024 22:13
@avdb13avdb13force-pushed theoauth-session branch 7 times, most recently from8147a40 to98ff557CompareNovember 21, 2024 03:07
@avdb13avdb13force-pushed theoauth-session branch 3 times, most recently from9d73cff to10d1077CompareNovember 24, 2024 07:30
@erlend-sh
Copy link
Contributor

Is this still blocked on#243 or does the merged#282 suffice?

@sugyan
Copy link
Owner

I think what is left in#243 and what this pull-request does are pretty much the same, but both have so many conflicts that it might be better to rework them....
I'm going to try to make a little more progress on#243.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
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
@avdb13@sugyan@erlend-sh
[8]ページ先頭
©2009-2025 Movatter.jp