После следующего коммита:
committed 02:29PM - 13 Dec 19 UTC
был добавлен жестко заданный список разрешённых протоколов, игнорирующий то, что настроено в административном интерфейсе:
# frozen_string_literal: true
module WildcardUrlChecker
VALID_PROTOCOLS = %w(http https discourse).freeze
def self.check_url(url, url_to_check)
return nil if !valid_url?(url_to_check)
escaped_url = Regexp.escape(url).sub("\\*", '\S*')
url_regex = Regexp.new("\\A#{escaped_url}\\z", 'i')
url_to_check.match(url_regex)
end
… что приводит к тому, что любая попытка создать новый API-ключ пользователя с fivem://accept-auth в качестве URI перенаправления завершается ошибкой 403 без какой-либо информации в /logs или на экране конечного пользователя.
5 лайков
Привет!
Спасибо, что сообщили нам об этом. Мы постараемся как можно скорее привлечь кого-нибудь для проверки.
2 лайка
sam
(Sam Saffron)
02.Январь.2020 02:22:58
11
Спасибо за сообщение об этой проблеме. Мы работаем над исправлением, чтобы автоматически добавлять fivem в белый список, если он обнаруживается в списке allowed_user_api_auth_redirects.
4 лайка
david
(David Taylor)
02.Январь.2020 11:51:32
13
Я открыл PR здесь:
master ← davidtaylorhq:wildcard-url-checker-protocol
merged 04:03PM - 02 Jan 20 UTC
This is required for people using apps with custom protocols. We still verify th… e entire URL (including protocol) against the site setting value.
Refactored wildcard_url_checker so that it always returns a boolean, rather than sometimes returning a regex match.
Мы проверяем весь URL (включая протокол) по списку настроек сайта, поэтому, думаю, отдельный белый список не нужен.
8 лайков
david
(David Taylor)
02.Январь.2020 16:53:07
14
Это уже слито. @blattersturm , если вы обновитесь до последней версии, проблема должна быть решена.
7 лайков
david
(David Taylor)
Закрыл(а) тему
06.Январь.2020 18:00:11
15
Эта тема была автоматически закрыта через 4 дня. Новые ответы больше не принимаются.