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

Стало плохо #19

Closed
umputun opened this issue Dec 8, 2018 · 17 comments
Closed

Стало плохо #19

umputun opened this issue Dec 8, 2018 · 17 comments

Comments

@umputun
Copy link
Member

umputun commented Dec 8, 2018

  1. не работает поехали
  2. выбранная новость доходит не всем
@Reeywhaar
Copy link
Collaborator

Я добавил пр #18, это наверное связано. Протестировал на https://rtnews.vyrtsev.com (направил его на https://news.radio-t.com/api/v1). У меня работает, "поехали", тема выбирается.

@Reeywhaar
Copy link
Collaborator

Reeywhaar commented Dec 8, 2018

Что-то странное происходит, stream.radio-t.com говорит "не в силах". Отваливается иногда.

@umputun
Copy link
Member Author

umputun commented Dec 8, 2018

не, это было результатом моих судорожных попыток на ходу все перезапустить. не помоголо

@umputun
Copy link
Member Author

umputun commented Dec 8, 2018

Я добавил пр #18, это наверное связано.

Связано? Т.е. оно post не могло сделеать а auth потому что теряло куку? Я не выходил вообще по внешним ссылкам и оно думало что я залогинен. В ином случае я бы не видел "поехали" вообще. Оно конкретно сломалось, я даже api проверил, все на вид правильно

@Reeywhaar
Copy link
Collaborator

Честно говоря, не знаю в чем на самом деле проблема. У меня все работает. Видимо есть какие-то различия между тестовым и продакшн апи. Можешь на https://rtnews.vyrtsev.com протестировать, все ли сейчас нормально? Там версия с localStorage. Осторожно, там не тестовый апи.

@umputun
Copy link
Member Author

umputun commented Dec 8, 2018

[Error] Blocked https://news.radio-t.com/api/v1/show/start?timestamp=1544311914803 from asking for credentials because it is a cross-origin request.

это в сафари. upd: в хроме тоже самое. Этого запроса я вообще не вижу на стороне сервера, что понятно - его не пропускает вообще. Но какого оно вдруг стало cross-origin ?

@umputun
Copy link
Member Author

umputun commented Dec 8, 2018

mxhzx-201812-08173845-koih1

@Reeywhaar
Copy link
Collaborator

Так, я понял что различие в том, что тестовый на http, а прод на https.
У сафари есть пунктик[1][2] чтобы при preflight request нужно чтобы в ответе был Origin в Access-Control-Allow-Headers от сервера.
Но это не точно. Можешь для точности завести https на jess.umputun.com, пожалуйста?

@umputun
Copy link
Member Author

umputun commented Dec 9, 2018

там похоже этот cross-origin результат упавшего POST. Я обновил версии с #18, починилось

@Reeywhaar
Copy link
Collaborator

Reeywhaar commented Dec 9, 2018

Странная ситуация все же. Я так понял залогиниться получилось, но кнопка "поехали" не работала. Чем реквест на допустим /news/reload отличается от /show/start для сафари? Загадка.

@umputun
Copy link
Member Author

umputun commented Dec 9, 2018

/show/start это запрос с авторизацией, на него и приходил отлуп

@Reeywhaar
Copy link
Collaborator

Сделал proxy_pass https://rtnews.vyrtsev.com/api/v1 -> http://jess.umputun.com:8780/api/v1
Буду смотреть.

@grayodesa

This comment was marked as abuse.

@umputun
Copy link
Member Author

umputun commented Dec 9, 2018

то, что у тебя работало подозреваю связанно с тем, что твоя UI news не была обновлена и старая версия работала. Я пробовал их 2х браузеров, закрывал/открывал, ничего не помогало пока не поменяли auth хранилище.

@Reeywhaar – я вспомнил, что я еще необычного сделал. Перед началом я почистил все куки и все из localstorage

@grayodesa

This comment was marked as abuse.

@Reeywhaar
Copy link
Collaborator

@umputun я не пойму, сейчас то все нормально или что-то еще не так? Мне все же интересно, почему кука удалялась, я не удалял, expire стоял на три года.

@umputun
Copy link
Member Author

umputun commented Dec 10, 2018

сейчас все нормально, да

@umputun umputun closed this as completed Dec 10, 2018
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