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

API redirects may cause UI to crash #547

Open
Rikuoja opened this issue Jan 18, 2018 · 1 comment
Open

API redirects may cause UI to crash #547

Rikuoja opened this issue Jan 18, 2018 · 1 comment

Comments

@Rikuoja
Copy link
Contributor

Rikuoja commented Jan 18, 2018

Steps to reproduce:

  1. Set the getSettings.js apiBaseUrl to http://api.hel.fi/kerrokantasi-test, which the server will redirect to https as intended.
  2. a) Try localhost:8086/hearings/list, the redirect will cause the list to hang. Or
    b) try logging in with the SSO, logging in is not completed. Or
    c) Try localhost:8086/talvipyoraily fullscreen and try POSTing several comments on the map. The comments are not reloaded after POSTing, and Failed to fetch appears.

There may be other urls that the UI will respond to similarly. I have no idea why some redirects will cause the UI to give Failed to fetch or an uncaught promise, while other redirects (the main page, the hearing page) work alright.

@dotanthanh
Copy link
Contributor

I cannot reproduce this. It seems to work just fine to me.

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

4 participants