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

Merge Telegram.Bot.Extensions.Passport #1089

Open
karb0f0s opened this issue Apr 29, 2022 · 3 comments
Open

Merge Telegram.Bot.Extensions.Passport #1089

karb0f0s opened this issue Apr 29, 2022 · 3 comments

Comments

@karb0f0s
Copy link
Member

Merge Telegram.Bot.Extensions.Passport to the main repo.

If I remember correctly, we put it into separate repo because at the time main repo used .Net Standard 1.3 and Passport required some crypto APIs from .Net Standard 2.0.

@karb0f0s
Copy link
Member Author

karb0f0s commented Apr 29, 2022

Also we can pull CheckAuthorization from Telegram.Bot.Extensions.LoginWidget - the same algorithm used to validate Web App data

@wiz0u
Copy link
Member

wiz0u commented Jul 10, 2024

For now I see no user demand for merging.
We would have to convert it to STJ as well.

Note: SetPassportDataErrorsRequest is already generated in main repo.

@wiz0u
Copy link
Member

wiz0u commented Aug 21, 2024

pull CheckAuthorization - the same algorithm used to validate Web App data

actually no, there is a subtle difference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants