cesko-digital / app

Komunitní aplikace Česko.Digital
https://app.cesko.digital
BSD 3-Clause "New" or "Revised" License
22 stars 39 forks source link

Problémy při přihlašování k webu přes Slack #802

Open zoul opened 1 year ago

zoul commented 1 year ago

Několik uživatelů nám nahlásilo, že po kliknutí na tlačítko Přihlásit dostanou tuhle chybu:

bug

(Klíčové slovo pro vyhledávání: invalid_team_for_non_distributed_app.)

Moje hypotéza je, že situace vzniká, když jsou přihlášeni k jinému Slack Workspace. U jedné uživatelky to bylo i patrné v URL na screenshotu – chybová hláška Slacku měla URL adresa-jejich-firmy.slack.com. Co s tím můžeme dělat? Chtěli bychom přihlášení „zamknout“ jen na náš workspace; jde to? Pokud to nepůjde vyřešit pořádně, pomohl by přinejmenším „rozcestník“, na kterém bychom jim vysvětlili, co se vlastně bude dít a co je potřeba k tomu, aby to fungovalo správně. (Viz související ticket #648.)

zoul commented 1 year ago

Poznámky:

Když pomažu cookies Slacku, tak mě to přihlašovací flow vyzve, abych zadal URL workspace:

signin

Z toho vůbec nemám radost, protože to je věc, kterou uživatelé vůbec nebudou vědět. Bylo by skvělé, kdyby to flow rovnou navedlo uživatele na správný workspace.

zoul commented 1 year ago

OK, mírný pokrok, v ce47113 jsem přidal parametr team, který by měl prosadit přihlašování k našemu workspace. Bohužel pokud uživatel není přihlášený k žádnému workspace, tak mu Slack opět ukáže ten dialog výše a žádá o URL workspace, viz dokumentaci:

If the user is not signed in yet, the user will be asked to specify a workspace to sign in to. That workspace will then be used as they complete the authorization flow, regardless of any team parameter you provided when the flow began.

Ale pokud je uživatel přihlášený do více workspaces, tak by tohle mohlo pomoct. Bohužel se mně nedaří reprodukovat tu situaci, kdy jsem přihlášený do víc workspaces a přihlášení mě pošle do špatného – vždycky mě to pošle správně do č.d, anebo rovnou do dialogu pro zadání URL workspace.

zoul commented 1 year ago

Mhm, tak už mám repro. Stačí být přihlášený do libovolného jiného workspace, ale ne do workspace Česko.Digital, a pokus o přihlášení skončí tou chybou invalid_team_for_non_distributed_app. Zkusil jsem reklamovat u Slacku, nevidím jediný rozumný důvod, aby se to takhle chovalo.

zoul commented 1 year ago

Ze Slacku se rychle ozvali, že je to bug, že o něm vědí a plánují ho opravit, ačkoliv neví kdy 🤞 Nechám tenhle ticket otevřený, dokud to nespraví.

zoul commented 11 months ago

Výhledově chceme úplně opustit přihlašování přes Slack a nahradit ho běžným přihlášením přes jméno a heslo.

zoul commented 1 month ago

Přihlášení přes Slack jsme vrátili do hry a toto se stále děje. Ještě jsem našel https://github.com/outline/outline/issues/1621, kde autor píše, že mu pomohlo distribuovat otevřeně tu přihlašovací aplikaci:

now it is giving the prompt to select workspace instead of failing if not signed in

Stojí to za pokus?

thierryskoda commented 2 weeks ago

@zoul Have you found a solution for that? I'm also stuck with the team parameter not working 👎 😞

zoul commented 1 week ago

@thierryskoda Unfortunately not. I did briefly consider trying to distribute our auth integration publicly, since the ticket linked above mentions it could solve the issue, but in the end we’ll probably hope for Slack to fix the issue soon(ish).