-
Notifications
You must be signed in to change notification settings - Fork 6
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
Comments
Я добавил пр #18, это наверное связано. Протестировал на https://rtnews.vyrtsev.com (направил его на https://news.radio-t.com/api/v1). У меня работает, "поехали", тема выбирается. |
Что-то странное происходит, stream.radio-t.com говорит "не в силах". Отваливается иногда. |
не, это было результатом моих судорожных попыток на ходу все перезапустить. не помоголо |
Связано? Т.е. оно post не могло сделеать а auth потому что теряло куку? Я не выходил вообще по внешним ссылкам и оно думало что я залогинен. В ином случае я бы не видел "поехали" вообще. Оно конкретно сломалось, я даже api проверил, все на вид правильно |
Честно говоря, не знаю в чем на самом деле проблема. У меня все работает. Видимо есть какие-то различия между тестовым и продакшн апи. Можешь на https://rtnews.vyrtsev.com протестировать, все ли сейчас нормально? Там версия с localStorage. Осторожно, там не тестовый апи. |
[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 ? |
там похоже этот cross-origin результат упавшего POST. Я обновил версии с #18, починилось |
Странная ситуация все же. Я так понял залогиниться получилось, но кнопка "поехали" не работала. Чем реквест на допустим |
/show/start это запрос с авторизацией, на него и приходил отлуп |
Сделал proxy_pass https://rtnews.vyrtsev.com/api/v1 -> http://jess.umputun.com:8780/api/v1 |
This comment was marked as abuse.
This comment was marked as abuse.
то, что у тебя работало подозреваю связанно с тем, что твоя UI news не была обновлена и старая версия работала. Я пробовал их 2х браузеров, закрывал/открывал, ничего не помогало пока не поменяли auth хранилище. @Reeywhaar – я вспомнил, что я еще необычного сделал. Перед началом я почистил все куки и все из localstorage |
This comment was marked as abuse.
This comment was marked as abuse.
@umputun я не пойму, сейчас то все нормально или что-то еще не так? Мне все же интересно, почему кука удалялась, я не удалял, expire стоял на три года. |
сейчас все нормально, да |
The text was updated successfully, but these errors were encountered: