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

Can I use OAuth2 Class directly? or Could you please consider exporting it? #230

Open
sjquant opened this issue Apr 18, 2021 · 5 comments
Open

Comments

@sjquant
Copy link

sjquant commented Apr 18, 2021

Hello, I really appreciate all your efforts to provide with this awesome library.
But, I have an issue. one of my browser (whale sidebar) doesn't work with popup, which makes it impossible to close popup.

So, I need to implement OAuth2's exchangeForToken on my own using information of redirected URL. And to do that, I need to copy and paste OAuth2 and other utils for that to work. It works well.

It will be really helpful if I can use OAuth2 class in this library, and think it would make this more expandable.

@dgrubelic
Copy link
Owner

Hi @sjquant , thanks for your feedback.

I'm currently working on refactoring entire library in TypeScript and can introduce some other architectural changes so I'll take your request into consideration.

Until that is done, I suggest you clone this repo and implement required changes for your project.

@dgrubelic
Copy link
Owner

Btw, can you provide a bit more information on what issues you have in that browser? Do you know what is causing the issue? Can you post some error logs?

@sjquant
Copy link
Author

sjquant commented Apr 19, 2021

Thanks for your fast reply. I'm really happy to hear that.

Btw, can you provide a bit more information on what issues you have in that browser? Do you know what is causing the issue? Can you post some error logs?

In Korea, there are browsers called 'Whale', and which browser provides kind of sub-browser called 'Whale Sidebar'.

image

On this browser, popup is considered just 'page change', not new window based on parent window. so, parent window loses its context. I know this is very special case, but it will be great to deal with this special case with on my own, because this library has provided with all functionality to do that.

There are no error logs, because it loses its parent's window context (parent window is replaced by popup).

@ajmas
Copy link

ajmas commented Dec 9, 2021

@sjquant if the auth used a non-pop-up would this resolve the issue for you?

@sjquant
Copy link
Author

sjquant commented Dec 10, 2021

@ajmas Yes, it woud be.

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

3 participants