better param parsing #36

coolaj86 · 2019-05-28T23:09:55Z

coolaj86 прокомментировал(а)

2019-05-28 23:09:55 +00:00

Владелец

Removing any and all of the following leading characters before parsing the params: #/?.

#access_token=x&foo=bar => access_token=x&foo=bar
#/?access_token=x&foo=bar => access_token=x&foo=bar
#/access_token=x&foo=bar => access_token=x&foo=bar
#?access_token=x&foo=bar => access_token=x&foo=bar

Removing any and all of the following leading characters before parsing the params: `#/?`. * `#access_token=x&foo=bar` => `access_token=x&foo=bar` * `#/?access_token=x&foo=bar` => `access_token=x&foo=bar` * `#/access_token=x&foo=bar` => `access_token=x&foo=bar` * `#?access_token=x&foo=bar` => `access_token=x&foo=bar`

Ghost одобрил(а) эти изменения 2019-05-29 01:49:48 +00:00

lib/html/js/app.js

						
				@ -4,3 +4,3 @@

				document.body.hidden = false;

				var hash = window.location.hash.substr(1);

				var hash = window.location.hash.replace(/^[\/#?]+/, '');

Ghost прокомментировал(а)

2019-05-29 01:49:42 +00:00

First-time contributor

Would #? to be safe, but I think inside a square bracket it's fine.

Would \#\? to be safe, but I think inside a square bracket it's fine.

coolaj86 прокомментировал(а)

2019-05-29 21:38:30 +00:00

Автор

Владелец

true on both accounts

coolaj86 закрыл этот запрос на слияние

2019-05-29 21:38:17 +00:00

coolaj86 прокомментировал(а)

2019-05-31 22:19:49 +00:00

Автор

Владелец

Thanks @mikegwhit

Войдите, чтобы присоединиться к обсуждению.

Нет рецензентов

Нет меток

feature

Нет этапа

Нет назначенных лиц

2 участников

Уведомления

Срок выполнения

Срок действия недействителен или находится за пределами допустимого диапазона. Пожалуйста, используйте формат 'гггг-мм-дд'.

Срок выполнения не установлен.

Зависимости

Зависимостей нет.

Ссылка: coolaj86/telebit.js#36