-
Notifications
You must be signed in to change notification settings - Fork 159
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't authorise #1843
Comments
Assuming you a bit messed up with formatting the issue message and both items under |
Excuse me for the late response, I wasn't able to check my mail. Edit: Thank you for an example of auth link. I have manually created a correct link by inserting my code_challenge and state there and frontier authorization page has been opened. Logged in, approved. So it looks like my Chrome doesn't understand a link and breaks him in two. Must be something with language settings... |
It's definitely not intended behavior. EDMarketConnector should open only one tab in order to auth, and it's URL should be as in the template I've specified. There are actually space between two parts of URL you're facing as opening as two tabs. It seems something between EDMarketConnector's call to webbrowser.open() and actually opening browser window treats space as separator between two URLs. This space is intended to encode to Other tools can be https://inara.cz and https://journal-limpet.com |
Thank you for your help. It must be something wrong with my Chrome. I will investigate it further. In the meantime, I will use a workaround with a manual link, it seems to work well. |
I would leave this issue as opened if next maintainer will want to look into actually encoding URL on EDMarketConnector side to prevent cases like this. Note that it mostly won't be addressed anytime soon as there is no active maintainer for now. |
Good idea! |
Same problem, cannot authorize. Tried to change browser from Chrome to Edge. Didn't work either. Hope to get a new update soon, cheers |
Looking through this, I'm as of yet unable to replicate this issue. however, when looking at the URL parameters I do see where it's created as: EDMarketConnector/companion.py Lines 395 to 404 in 21268d5
It's this Hesitant to make code changes without confirming this is the source. |
I ended up thinking that it's a problem with space in |
Replaces vague space with properly encoded %20. This SHOULD solve the issue, but I can't replicate the original issue.
Stepped away from Chrome and reinstalled Windows in the meantime. It works fine atm, thank you, guys, for fixing stuff and keep it alive! |
Please complete the following information:
%TEMP%\EDMarketConnector.log
from immediately after the bug occurs (re-running the application overwrites this file).%TEMP%\EDMarketConnector\EDMarketConnector-debug.log
. See Debug Log File. NB: If you don't have this log file then you're not running the latest version of the application and should update first to see if we already fixed the bug you're reporting.EDMarketConnector.log
EDMarketConnector-debug.log
Describe the bug
After launch 2 tabs are opened in chrome:
App is stuck on 'Logging in...'
To Reproduce
Start EDMC
Expected behavior
One of the tabs should be a Frontier authorization page? As far as I can remember I never saw that page.
Screenshots
Additional context
Launching with EDMarketConnector - localserver-auth.bat does the same thing.
Reinstall doesn't help either
The text was updated successfully, but these errors were encountered: