search

Traewelling / traewelling

Free check-in service to log your public transit journeys
https://traewelling.de
GNU Affero General Public License v3.0
225 stars 43 forks source link

Login redirects to "/notifications/latest" #28

Closed aledjones closed 4 years ago

aledjones commented 4 years ago

Describe the bug After login, the user gets redirected to /notifications/latest. This results in the delivery of the corresponding JSON file.

To Reproduce Steps to reproduce the behavior:

  1. Go to login page, unauthenticated.
  2. Login with credentials (tested with mail login, not social)
  3. you get served a JSON file.

Expected behavior After login, the user gets redirected to his dashboard.

Desktop (please complete the following information):

Additional context Commit: 1c55723227b1399d623ad3f4906420eebd8bb130

HerrLevin commented 4 years ago

Are you in dev or prod environment?

Am So, 1. Mär 2020, um 12:27, schrieb Jonas Möller:

Describe the bug After login, the user gets redirected to /notifications/latest. This results in the delivery of the corresponding JSON file.

To Reproduce Steps to reproduce the behavior:

  1. Go to login page, unauthenticated.
  2. Login with credentials (tested with mail login, not social)
  3. you get served a JSON file.

Expected behavior After login, the user gets redirected to his dashboard.

Desktop (please complete the following information):

  • OS: Debian 10 Buster
  • Browser: Mozilla Firefox
  • Version: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Firefox/68.0

Additional context Commit: 1c55723227b1399d623ad3f4906420eebd8bb130 https://github.com/Traewelling/traewelling/commit/1c55723227b1399d623ad3f4906420eebd8bb130

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Traewelling/traewelling/issues/28?email_source=notifications&email_token=AAJVRNT465PF6YN3J6XS3IDRFJBBRA5CNFSM4K7D7GOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRPMFOA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJVRNR7SQFXIRS6C4AQ7N3RFJBBRANCNFSM4K7D7GOA.

aledjones commented 4 years ago

dev

Should I post my .env file contents for clarification?

HerrLevin commented 4 years ago

No, I have an idea.

There was a quickfix pre-release to fix iOS PWA-Problems. Could you please start the application as prod?

Am So, 1. Mär 2020, um 12:29, schrieb Jonas Möller:

dev env

Should I post my .env file contents for clarification?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Traewelling/traewelling/issues/28?email_source=notifications&email_token=AAJVRNUSHD64QUDWJNOAP3TRFJBI3A5CNFSM4K7D7GOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENM4PZQ#issuecomment-593086438, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJVRNXVSQNP3QLIPV7FMFLRFJBI3ANCNFSM4K7D7GOA.

aledjones commented 4 years ago

Same result with prod environment

jeyemwey commented 4 years ago

Hey @aledjones - does this bug still occur?

aledjones commented 4 years ago

Bug does still occur. +1 other bug, but I'll open an extra issue for that.

HerrLevin commented 4 years ago

I found the source of the error! It seems to be the fetch -method in resources/js/components/notifications-board.js .